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 <[email protected]>
  • Loading branch information
ShGKme authored and backportbot[bot] committed Aug 22, 2024
1 parent f7c8b5b commit 7c94cfc
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 7c94cfc

Please sign in to comment.