-
-
Notifications
You must be signed in to change notification settings - Fork 301
[Remove Vuetify from Studio] Main navigation in Channels #5642
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
Prashant-thakur77
wants to merge
103
commits into
learningequality:unstable
Choose a base branch
from
Prashant-thakur77:remove-vuetify-navigation-5369
base: unstable
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
103 commits
Select commit
Hold shift + click to select a range
aa65fef
Mkaing the studionavigationtab
Prashant-thakur77 c928f12
Updated
Prashant-thakur77 54ac677
FIX: resolved the merge conflicts
Prashant-thakur77 a389d2a
Updated
Prashant-thakur77 337fac5
Updated
Prashant-thakur77 1dc17d7
Updated
Prashant-thakur77 b77bf6e
Update
Prashant-thakur77 ca26cd6
Update
Prashant-thakur77 1976faf
Update styles
Prashant-thakur77 6e16435
Updated sidepanel styles
Prashant-thakur77 e90db43
Updated sidepanel styles
Prashant-thakur77 3d373ba
Updated sidepanel styles
Prashant-thakur77 9e2141f
Updated sidepanel styles
Prashant-thakur77 84ab07e
Updated sidepanel styles
Prashant-thakur77 e3b7b1a
Updated sidepanel styles
Prashant-thakur77 f20ad6b
Updated sidepanel styles
Prashant-thakur77 7bf5c5c
Updated styles
Prashant-thakur77 dff36ab
Updated
Prashant-thakur77 3d2380a
Updated styles for tabs
Prashant-thakur77 f2a24d3
Updated styles for tabs
Prashant-thakur77 a7a6557
Updated styles for tabs
Prashant-thakur77 838ce74
Updated styles for tabs
Prashant-thakur77 7b18f88
Updated styles of sidepanel
Prashant-thakur77 57cda47
Updated styles of sidepanel
Prashant-thakur77 16281d1
Updated sidepanel styles
Prashant-thakur77 71ca32b
Updated sidepanel styles
Prashant-thakur77 7932532
Updated sidepanel styles
Prashant-thakur77 963e121
Updated sidepanel styles
Prashant-thakur77 9b2e1ad
Added scrollable funcality to tabs
Prashant-thakur77 9a0d4e3
Updated the link design
Prashant-thakur77 319f42d
Updated the link design
Prashant-thakur77 8ff2a48
Updated the link design
Prashant-thakur77 1720fd4
Updated the link design
Prashant-thakur77 5e32e1b
Updated the link design
Prashant-thakur77 00b7cce
Updated the link design
Prashant-thakur77 6ef271d
Updated the title styles
Prashant-thakur77 a3fd6c8
Updated the title styles
Prashant-thakur77 f3cb217
Updated the sliding indicator
Prashant-thakur77 5aad0d4
Updated the sliding indicator
Prashant-thakur77 9a7b09a
Updated class anmes
Prashant-thakur77 2b42af3
Updated the tabs container
Prashant-thakur77 32bb4cb
Sliding approach
Prashant-thakur77 a1344f7
Sliding approach
Prashant-thakur77 8b431fd
old approach after review
Prashant-thakur77 6a61c05
updated code for tabs
Prashant-thakur77 591b186
Updated menu styles
Prashant-thakur77 9cb20be
Added StudioPanelOption.vue
Prashant-thakur77 7ae64b0
Added StudioPanelOption.vue
Prashant-thakur77 22f7f86
Added StudioPanelOption.vue
Prashant-thakur77 9bfee50
Added StudioPanelOption.vue
Prashant-thakur77 33e6d5e
Added StudioPanelOption.vue
Prashant-thakur77 e8cefe5
Updated styles
Prashant-thakur77 03f4de0
Updated the skipnavigationlink functionality
Prashant-thakur77 ead0841
Removed the padding added to the sie panle modal and rather added tha…
Prashant-thakur77 79290ed
updatedstyles
Prashant-thakur77 d02965e
updatedstyles
Prashant-thakur77 394597e
Updated Catalogfiltercontent to handle padding wihout sidepanelmodal
Prashant-thakur77 31c946f
Updated boolean
Prashant-thakur77 e68dab5
Updated the tabtrack method to studionavigation
Prashant-thakur77 fd5d3e3
Updated the tabtrack method to studionavigation
Prashant-thakur77 ada5e5e
finalising
Prashant-thakur77 489626a
Test file updated
Prashant-thakur77 492d824
Test file created
Prashant-thakur77 533ade1
Updated test files
Prashant-thakur77 36f3ccb
Updated strings
Prashant-thakur77 bfd3abb
Test files
Prashant-thakur77 32ec1ac
Final updates
Prashant-thakur77 dd2ecdd
Final updates
Prashant-thakur77 251656c
Final updates
Prashant-thakur77 0f80c86
Updated class names
Prashant-thakur77 5a038c0
precommit
Prashant-thakur77 0d03d85
Added aria-label to the nav
Prashant-thakur77 a618088
Updated
Prashant-thakur77 e36c572
updated
Prashant-thakur77 d281fc6
resolving review
Prashant-thakur77 ce61aa0
used ktexttruncaotr
Prashant-thakur77 482aad7
aligned the ktexttruncator with ktoolbar
Prashant-thakur77 390931d
removed aria-live
Prashant-thakur77 74d420c
updated
Prashant-thakur77 fafbcdd
don
Prashant-thakur77 2f23a0c
decresd offsst
Prashant-thakur77 03dec67
updated
Prashant-thakur77 ceb8e12
vreated a diff file to sidepanel to manage codebas
Prashant-thakur77 9947cef
UPdated with reviews
Prashant-thakur77 b6ce72b
Removed magic stirngs
Prashant-thakur77 1426c39
done
Prashant-thakur77 aec064f
Made updates
Prashant-thakur77 9a6d4f5
Made updates
Prashant-thakur77 b58b2ba
Made updates
Prashant-thakur77 26f70b7
Made updates
Prashant-thakur77 5145e53
Made updates
Prashant-thakur77 d29a725
corrected test suits
Prashant-thakur77 6b381e0
[pre-commit.ci lite] apply automatic fixes
pre-commit-ci-lite[bot] fb4217e
FIX: Resolve merge conflicts and lint errors in ChannelListIndex.vue
Prashant-thakur77 70d6d6b
FEAT: Add notification indicators to StudioNavigation and SidePanel
Prashant-thakur77 e7d258c
FIX: Move NotificationsModal into explicit default slot
Prashant-thakur77 9080f8b
Resolved merge issues
Prashant-thakur77 091ee4e
Fix: Fixed kdropdown size
Prashant-thakur77 b6ea656
Fixing Kdropdown sizes
Prashant-thakur77 8531895
Fixing Kdropdown sizes
Prashant-thakur77 b6b0726
Fix: Resolved rtl issues in the red dot usage for notification
Prashant-thakur77 3870b87
Resolved some indentaion
Prashant-thakur77 27a1f42
Added transition to Vbadge
Prashant-thakur77 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
89 changes: 89 additions & 0 deletions
89
...entcuration/contentcuration/frontend/shared/views/StudioNavigation/SkipNavigationLink.vue
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,89 @@ | ||
| <template> | ||
|
|
||
| <div class="skip-nav-link"> | ||
| <KButton | ||
| ref="button" | ||
| :style="linkStyles" | ||
| appearance="basic-link" | ||
| :text="$tr('skipToMainContentAction')" | ||
| @click="handleClickSkipLink" | ||
| /> | ||
| </div> | ||
|
|
||
| </template> | ||
|
|
||
|
|
||
| <script> | ||
|
|
||
| export default { | ||
| name: 'SkipNavigationLink', | ||
| computed: { | ||
| linkStyles() { | ||
| return { | ||
| backgroundColor: this.$themeTokens.surface + ' !important', | ||
| }; | ||
| }, | ||
| }, | ||
| methods: { | ||
| handleClickSkipLink() { | ||
| // Every page where this is supposed to work needs to have a top-level | ||
| // element with 'role' and 'id' attribute equal to 'main' and 'tabindex= -1'. | ||
| // If it doesn't have one, clicking this link is a noop, but will re-focus itself | ||
| // as a convenience (in case main div is still loading). | ||
| const mainEl = document.getElementById('main'); | ||
| if (mainEl) { | ||
| // If it exists, actually target and focus on the main header | ||
| const header = mainEl.querySelector('h1'); | ||
| if (header) { | ||
| // HACK: Need to set its tabindex attribute on the fly to get tab behavior | ||
| header.setAttribute('tabindex', -1); | ||
| header.focus(); | ||
| } else { | ||
| mainEl.focus(); | ||
| } | ||
| } else { | ||
| // NOTE: the button retains focus, but loses :focus styling after hitting "Enter" | ||
| // TODO: look into theme input modality to see if we can get consistent | ||
| // styling when in keyboard modality | ||
| this.$refs.button.$el.focus(); | ||
| } | ||
| }, | ||
| }, | ||
| $trs: { | ||
| skipToMainContentAction: { | ||
| message: 'Skip to main content', | ||
| context: | ||
| 'Button label for the hamburger menu in the top left corner. Accessible only to those who use screen readers and other assistive technology (AT).', | ||
| }, | ||
| }, | ||
| }; | ||
|
|
||
| </script> | ||
|
|
||
|
|
||
| <style lang="scss" scoped> | ||
|
|
||
| @import '~kolibri-design-system/lib/styles/definitions'; | ||
|
|
||
| .skip-nav-link { | ||
| position: relative; | ||
| z-index: 2; | ||
| } | ||
|
|
||
| .skip-nav-link a { | ||
| @extend %dropshadow-2dp; | ||
|
|
||
| position: absolute; | ||
| left: -1000px; | ||
| padding: 8px 16px; | ||
| font-size: 14px; | ||
| outline-offset: 0 !important; | ||
|
|
||
| &:focus { | ||
| top: 8px; | ||
| left: 8px; | ||
| z-index: 2; | ||
| } | ||
| } | ||
|
|
||
| </style> |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MisRob This prop is added here to deal with the full width functionality of sidepanelmodel