Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Commit 6d4d8a7

Browse files
committed
💄 Clean up vertical tabs
1 parent 593c615 commit 6d4d8a7

File tree

2 files changed

+21
-16
lines changed

2 files changed

+21
-16
lines changed

src/browser/themes/pulse/addons/shared.css

+5-5
Original file line numberDiff line numberDiff line change
@@ -87,6 +87,11 @@ tab.tabbrowser-tab:nth-child(1) {
8787
right: var(--tab-rounding-size-neg);
8888
}
8989

90+
/* This overrides our settings for the proton theme */
91+
#browser #TabsToolbar .tab-background {
92+
height: var(--tab-min-height);
93+
}
94+
9095
#tabbrowser-arrowscrollbox .tabbrowser-tab {
9196
/* Specifies the padding to the left and right of the tab */
9297
padding-inline: var(--tab-padding-inline) !important;
@@ -103,11 +108,6 @@ tab.tabbrowser-tab:nth-child(1) {
103108
border-radius: var(--toolbarbutton-border-radius) !important;
104109
}
105110

106-
/* Vertical tab new button stuff */
107-
#browser #TabsToolbar #new-tab-button {
108-
margin: 8px;
109-
}
110-
111111
#verticaltabs-splitter {
112112
background-color: var(--tab-selected-bgcolor, var(--toolbar-bgcolor));
113113
background-image: var(--tab-selected-bgimage, var(--toolbar-bgimage));

src/browser/themes/pulse/vertical_tabs.css

+16-11
Original file line numberDiff line numberDiff line change
@@ -71,11 +71,16 @@
7171

7272
/* Stops the tabs from getting squished or stretched */
7373
min-height: unset;
74+
height: var(--tab-min-height);
7475
max-height: var(--tab-min-height);
7576

7677
transition: min-height 150ms ease-in-out, max-height 150ms ease-in-out;
7778
}
7879

80+
#browser #TabsToolbar .tab-background {
81+
height: calc(var(--tab-min-height) - 2px);
82+
}
83+
7984
/* Work around for tab closing animation whilst in vertical tabs. Without this
8085
the clean up code (created by 608589) will not work correctly */
8186
/* TODO: This causes tabs to stick around for longer and trigger a clean up
@@ -115,6 +120,17 @@
115120

116121
#browser #new-tab-button {
117122
display: inherit !important;
123+
appearance: inherit !important;
124+
125+
border-radius: var(--tab-border-radius);
126+
margin: 2px;
127+
padding: var(--inline-tab-padding);
128+
129+
height: calc(var(--tab-min-height) - 2px);
130+
}
131+
132+
#browser #new-tab-button:hover {
133+
background: var(--toolbarbutton-hover-background);
118134
}
119135

120136
#browser #tabbrowser-tabs {
@@ -136,14 +152,3 @@
136152
:root[sizemode='fullscreen'] #browser #TabsToolbar {
137153
display: none;
138154
}
139-
140-
#new-tab-button {
141-
appearance: inherit !important;
142-
border-radius: 5px;
143-
margin-left: 5px !important;
144-
margin-right: 5px !important;
145-
}
146-
147-
#new-tab-button:hover {
148-
background: var(--toolbarbutton-hover-background);
149-
}

0 commit comments

Comments
 (0)