Skip to content

Commit

Permalink
Fix: Hidden - Remove padding at Menubar is shown #460
Browse files Browse the repository at this point in the history
  • Loading branch information
black7375 committed Sep 2, 2022
1 parent 5663d22 commit 8c1f844
Show file tree
Hide file tree
Showing 7 changed files with 63 additions and 2 deletions.
34 changes: 34 additions & 0 deletions __tests__/_has.test.scss
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
@use "true" as *;
@use "example" as *;
@use "../src/utils/has" as *;

@include test-module("`:has()` selector test [mix]") {
@include test("exist `:has()`") {
@include assert {
@include output {
@include Has {
@include example;
}
}
@include expect {
@supports selector(:has(a)) {
@include example;
}
}
}
}
@include test("don't exist `:has()`") {
@include assert {
@include output {
@include NotHas {
@include example;
}
}
@include expect {
@supports not selector(:has(a)) {
@include example;
}
}
}
}
}
6 changes: 6 additions & 0 deletions css/leptonChrome.css
Original file line number Diff line number Diff line change
Expand Up @@ -4637,6 +4637,12 @@
}
}
}
@supports selector(:has(a)) {
#navigator-toolbox:has(#toolbar-menubar[autohide="false"]) {
--uc-window-drag-space-pre: 0px;
--uc-window-control-space: 0px;
}
}
}
@supports -moz-bool-pref("userChrome.tabbar.on_bottom") or -moz-bool-pref("userChrome.tabbar.one_liner") or -moz-bool-pref(
"userChrome.hidden.tabbar"
Expand Down
2 changes: 1 addition & 1 deletion src/autohide/_bookmarkbar.scss
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
will-change: margin-bottom, opacity;

@include Option("userChrome.autohide.toolbar_overlap") {
@supports not selector(:has(a)) {
@include NotHas {
&[collapsed="true"] {
visibility: visible !important;
max-height: unset !important;
Expand Down
2 changes: 1 addition & 1 deletion src/autohide/_common.scss
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
--uc-navbar-height: 0px;
}
}
@supports selector(:has(a)) {
@include Has {
#navigator-toolbox:has(#PersonalToolbar[collapsed="true"]) {
--uc-bm-height: 0px;
}
Expand Down
1 change: 1 addition & 0 deletions src/leptonChrome.scss
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@
@use "utils/native_menu" as *;
@use "utils/one_liner" as *;
@use "utils/window_control" as *;
@use "utils/has" as *;
@use "utils/proton_elements" as Proton;
@use "sass:selector";

Expand Down
7 changes: 7 additions & 0 deletions src/tabbar/layout/_window_control_size.scss
Original file line number Diff line number Diff line change
Expand Up @@ -101,3 +101,10 @@
}
}
}

@include Has {
#navigator-toolbox:has(#toolbar-menubar[autohide="false"]) {
@include _remove_spacer_pre;
@include _remove_spacer_post;
}
}
13 changes: 13 additions & 0 deletions src/utils/_has.scss
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
// Need `:has selector`

@mixin Has() {
@supports selector(:has(a)) {
@content;
}
}

@mixin NotHas() {
@supports not selector(:has(a)) {
@content;
}
}

0 comments on commit 8c1f844

Please sign in to comment.