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

'autohide_bookmarks_and_main_toolbars.css' has a problem since 133 #482

Open
ilxtx opened this issue Dec 5, 2024 · 9 comments
Open

'autohide_bookmarks_and_main_toolbars.css' has a problem since 133 #482

ilxtx opened this issue Dec 5, 2024 · 9 comments

Comments

@ilxtx
Copy link

ilxtx commented Dec 5, 2024

https://github.com/MrOtherGuy/firefox-csshacks/tree/master/chrome/autohide_bookmarks_and_main_toolbars.css

Hello, from version 133 to the latest version 134, there is a small problem with this CSS. When clicking the button in the address bar and hovering over the pop-up window that appears, the address bar and bookmark toolbar will be automatically hidden, instead of being displayed continuously. See the picture below for details:

autohide_bookmarks_and_main_toolbars css-issus

I like this CSS very much and hope you can fix this problem, thank you very much.

@MrOtherGuy
Copy link
Owner

This is potentially an easy fix, the only complication is that if I do it then the toolbars might also show in some unexpected circumstances which wouldn't be great.

@ilxtx
Copy link
Author

ilxtx commented Dec 6, 2024

This is potentially an easy fix, the only complication is that if I do it then the toolbars might also show in some unexpected circumstances which wouldn't be great.

Thank you for your reply.

Yes, there is a situation you mentioned. I hope there can be a perfect solution. This CSS is perfect in version 132 and before, I don't know why this issue has appeared since version 133.

Thank you again for this very practical project.

@MrOtherGuy
Copy link
Owner

I have now changed a few selectors that should hopefully make more of the panels to keep toolbars visible, hopefully without causing annoying disturbances.

@ilxtx
Copy link
Author

ilxtx commented Dec 8, 2024

I have now changed a few selectors that should hopefully make more of the panels to keep toolbars visible, hopefully without causing annoying disturbances.

Thank you very much. It's very perfect and has passed testing in versions 132 and 134.

@konsulas
Copy link

konsulas commented Dec 8, 2024

I have another issue since version 133.
I use "autohide_bookmarks_and_main_toolbars.css" with couple modified settings:

#navigator-toolbox{
--uc-bm-padding: 4px; /* Vertical padding to be applied to bookmarks /
--uc-bm-height: calc(20px + 2 * var(--uc-bm-padding)); /
Might need to adjust if the toolbar has other buttons /
--uc-navbar-height: -40px; /
navbar is main toolbar. Use negative value /
--uc-autohide-toolbar-delay: 300ms; /
The toolbar is hidden after 0.6s */

and

/* Show when cursor is over the toolbar area */
#navigator-toolbox:hover > .browser-toolbar{
transition-delay: 1200ms !important;
transform: rotateX(0);

Now when mouse pointer is hovering over tab bar, address bar is opening immediately without delay and in very weird way

1

and only after 1200ms delay bookmark toolbar shows up.

2

@MrOtherGuy
Copy link
Owner

Urlbar needs to be transformed separately ,so you'll have to also modify the delay for urlbar here on line 74.

@konsulas
Copy link

konsulas commented Dec 8, 2024

Urlbar needs to be transformed separately ,so you'll have to also modify the delay for urlbar here on line 74.

Working! Thanks.

@paulscholes0258
Copy link

@MrOtherGuy
Hi Bro. how to use autohide_bookmarks_and_main_toolbars.css + autohide_tabstoolbar_v2.css in 1 css.
each of both separate css working good but when I copy combine ( copy text) 2 css in one, it's not working. thanks

@MrOtherGuy
Copy link
Owner

Simply don't. Since you want to autohide all toolbars you should use autohide_toolbox.css instead.

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

No branches or pull requests

4 participants