Skip to content

Commit

Permalink
fix(NcAppNavigation): keep border only on mobile
Browse files Browse the repository at this point in the history
Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
  • Loading branch information
ShGKme authored and susnux committed Aug 29, 2024
1 parent 749fcaf commit baccadb
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion src/components/NcAppNavigation/NcAppNavigation.vue
Original file line number Diff line number Diff line change
Expand Up @@ -339,7 +339,6 @@ export default {
background-color: var(--color-main-background-blur, var(--color-main-background));
-webkit-backdrop-filter: var(--filter-background-blur, none);
backdrop-filter: var(--filter-background-blur, none);
border-inline-end: 1px solid var(--color-border);
&--close {
margin-left: calc(-1 * min($navigation-width, var(--app-navigation-max-width)));
Expand Down Expand Up @@ -378,10 +377,18 @@ export default {
}
}
// Add extra border for high contrast mode
[data-themes*="highcontrast"] {
.app-navigation {
border-inline-end: 1px solid var(--color-border);
}
}
// When on mobile, we make the navigation slide over the NcAppContent
@media only screen and (max-width: $breakpoint-mobile) {
.app-navigation {
position: absolute;
border-inline-end: 1px solid var(--color-border);
}
}
Expand Down

0 comments on commit baccadb

Please sign in to comment.