Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add drag space above tabs #210

Closed
failsafe42 opened this issue Sep 2, 2021 · 2 comments
Closed

Add drag space above tabs #210

failsafe42 opened this issue Sep 2, 2021 · 2 comments
Labels
Class::Padding Padding and margin related layout Component::Tabbar Tab or tabbar Issue::Enhancement New feature or request

Comments

@failsafe42
Copy link

Before proton Firefox let you add extra drag space above the tabs, but now that option is gone.

Could you add back the option to have that drag space?

@failsafe42 failsafe42 added the Issue::Enhancement New feature or request label Sep 2, 2021
@black7375 black7375 added the Class::Padding Padding and margin related layout label Sep 3, 2021
@black7375
Copy link
Owner

black7375 commented Sep 3, 2021

Just log (Firefox v89's css)

:root {
  --space-above-tabbar: 0px;
}
:root[extradragspace][tabsintitlebar]:not([inFullscreen]) {
    --space-above-tabbar: 8px;
}
:root[tabsintitlebar][sizemode="normal"] #toolbar-menubar[autohide="true"] {
  height: calc(var(--tab-min-height) + var(--space-above-tabbar)); 
}

/* Add extra space to titlebar for dragging */
:root[sizemode="normal"][chromehidden~="menubar"] #TabsToolbar > .toolbar-items,
:root[sizemode="normal"] #toolbar-menubar[autohide="true"][inactive] + #TabsToolbar > .toolbar-items {
  padding-top: var(--space-above-tabbar);
}


#tabbrowser-tabs {
  --tabs-top-border-width: 0px;
}
#toolbar-menubar:not([autohide=true]) + #TabsToolbar,
:root:not([tabsintitlebar]),
:root[extradragspace] {
  --tabs-top-border-width: 1px;
}
:root[extradragspace] .tab-background {
  border-top-style: solid;
}
:root[tabsintitlebar]:not([extradragspace]) #toolbar-menubar[autohide=true] + #TabsToolbar .tabbrowser-tab::after,
/* Show full height tab separators on hover and multiselection. */
.tabbrowser-tab:hover::after,
#tabbrowser-tabs:not([movingtab]) > #tabbrowser-arrowscrollbox > .tabbrowser-tab[beforehovered]::after,
.tabbrowser-tab[multiselected]::after,
#tabbrowser-tabs:not([movingtab]) > #tabbrowser-arrowscrollbox > .tabbrowser-tab[before-multiselected]::after {
  margin-top: var(--tabs-top-border-width);
  margin-bottom: 0;
}

@black7375 black7375 added Component::Toolbar Toolbar or navbar Component::Tabbar Tab or tabbar and removed Component::Toolbar Toolbar or navbar labels Sep 3, 2021
@black7375
Copy link
Owner

Will be supported drag space.

/* for Extra Drag Space */
:root {
--space-above-tabbar: 0px; /* If you want drag space, set to 8px */
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Class::Padding Padding and margin related layout Component::Tabbar Tab or tabbar Issue::Enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants