Skip to content

Commit

Permalink
Merge pull request #6008 from nextcloud-libraries/fix/nc-app-navigati…
Browse files Browse the repository at this point in the history
…on--border-right

fix(NcAppNavigation): keep border only on mobile
  • Loading branch information
susnux authored Aug 22, 2024
2 parents f84ff80 + e675505 commit 73476cb
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 @@ -342,7 +342,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 @@ -389,10 +388,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 73476cb

Please sign in to comment.