Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Arch linux: Scroll bar appears over tabs when they fill the tab bar #2599

Closed
jkav77 opened this issue Jul 20, 2016 · 8 comments
Closed

Arch linux: Scroll bar appears over tabs when they fill the tab bar #2599

jkav77 opened this issue Jul 20, 2016 · 8 comments

Comments

@jkav77
Copy link
Contributor

jkav77 commented Jul 20, 2016

My search skills may have failed me, but I did not find this bug. I apologize if this is a duplicate.

Problem: After opening enough tabs (depends on window size) to fill the tab bar, a horizontal scroll bar appears that conceals the lower ~75% of the tabs. I am still barely able to switch tabs by clicking on the very top of the tab, but it is difficult to see which tab is which. The scroll bar is always "full", meaning it cannot actually scroll left or right no matter how many tabs there are. Closing or pinning tabs until they no longer fill the available space causes the scroll bar to disappear.

Expected behavior: The scroll functionality should not conceal the tabs, and should effectively scroll to tabs or tab groups that are not currently visible.

Platform: Arch Linux

Brave Version: v0.11.0

Steps to reproduce:
1. Open brave
2. Open multiple tabs until they fill the available space in the tab bar.
3. Observe the scroll bar covering the tabs.
4. Close tabs until the scroll bar disappears.

2016-07-19-212108_1148x1041_scrot

@bbondy
Copy link
Member

bbondy commented Jul 20, 2016

PR wanted for this from someone who is running Arch Linux.

@bbondy
Copy link
Member

bbondy commented Jul 20, 2016

It should be a pretty small CSS tweak maybe with overflow-x: clip or something like that

@bbondy bbondy changed the title Scroll bar appears over tabs when they fill the tab bar Arch linux: Scroll bar appears over tabs when they fill the tab bar Jul 20, 2016
@babolivier
Copy link

babolivier commented Jul 20, 2016

Archlinux user here, I remember having this issue a while ago (even opened #1403), and didn't encountered it after that (even now, v0.11.0). I tried to reproduce it by pinning as much tabs as you have, then playing with the window's width, but couldn't:

capture d ecran de 2016-07-20 22-08-16

@dangerginger How did you install Brave? AUR? If so, which package? (I use the brave-bin one, maybe with the one you're using it behaves differently because of the install procedure)

@jkav77
Copy link
Contributor Author

jkav77 commented Jul 21, 2016

@babolivier, I initially installed the brave package from the AUR. As you suggested, I removed that and tried the brave-bin package but got the same behavior. I deleted the ~/.config/brave directory and restarted the browser, but also no change. I experience the same behavior in both i3 (tiling) and lxde.

@bbondy, I can look into fixing this but it would be my first PR. If you give me a hint on where I should look into this it would help me get started.

Thanks

@bbondy
Copy link
Member

bbondy commented Jul 21, 2016

You can fix this by using the built in browser level dev tools.
The shortcut of this is changing soon, so if you see it stop working just check the recent commit history.
But for now you can use Control+Alt+J. Then you can use the DOM inspector on the tab bar and try to tweak it to fix just like for a web page.

Once you find the styles that need to change just git grep term for where to change it.

The README has information about first time setup.

@jkav77
Copy link
Contributor Author

jkav77 commented Jul 21, 2016

Great. I will look into this over the weekend. Thanks for pointing me in
the right direction.

On Thu, Jul 21, 2016, 07:03 Brian R. Bondy notifications@github.com wrote:

You can fix this by using the built in browser level dev tools.
The shortcut of this is changing soon, so if you see it stop working just
check the recent commit history.
But for now you can use Control+Alt+J. Then you can use the DOM inspector
on the tab bar and try to tweak it to fix just like for a web page.

Once you find the styles that need to change just git grep term for where
to change it.

The README has information about first time setup.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#2599 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOKj9-NASz34z00ZdaQDWV82WE3uNjTpks5qX3v8gaJpZM4JQamc
.

@bbondy
Copy link
Member

bbondy commented Jul 21, 2016

awesome, thanks @dangerginger!

@jkav77
Copy link
Contributor Author

jkav77 commented Jul 21, 2016

I found the CSS selector using the method you described and sent my first ever PR. You can tell it's the first one because I unfortunately named it 'patch-1'. Sorry about that. Not very descriptive.

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

No branches or pull requests

4 participants