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

Fix: tab bar height, new tab button alignment and multi-account containers colored bar #327

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

M0kh
Copy link

@M0kh M0kh commented Aug 13, 2022

Edit: For all fixes in other issues and pull requests you can check out my fork of this repo here

The tab bar height is fixed by editing the margin:

In chrome\tabbar\tabbar.css

change the following:

Line 129:

from:
margin-top: 4px !important;

to:
margin-top: -4px !important;

2022-08-13 14_22_45-NVIDIA GeForce Overlay DT

The new tab button will be misaligned, to fix:

Line 492:

from:
margin-inline-start: 6px !important;

to:
margin-inline-start: 4px !important;
margin-bottom: 4px !important;
margin-top: 4px !important;

The tab before:
2022-08-13 13_45_52-NVIDIA GeForce Overlay DT

The tab after:
2022-08-13 13_46_37-NVIDIA GeForce Overlay DT

If you find any issue in your browser you can edit the margins to your liking.

If you use multi-account containers you will see that the colored bar at the bottom of the tab is either gone or it's gone when switching to another tab, to fix:

In chrome\userChrome.css

add the following:

/*move multi-account containers color to the bottom of the tab*/
.tab-context-line {
  margin: var(--tab-min-height) var(--inline-tab-padding) 0 !important;
}

In chrome\tabbar\tabbar.css

change the following:

Line 529:

from:
height: 2px !important;

to:
height: 6px !important;

Line 546:

from:
bottom: calc(100% - 9px) !important;

to:
bottom: calc(90% - 9px) !important;

Before:
2022-08-13 13_51_08-NVIDIA GeForce Overlay DT

After:
2022-08-13 13_53_24-NVIDIA GeForce Overlay DT

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant