MrOtherGuy / firefox-csshacks

Collection of userstyles affecting the browser
Mozilla Public License 2.0
3.19k stars 312 forks source link

One weird bug with "window_control_placeholder_support.css" #388

Closed datguypiko closed 1 month ago

datguypiko commented 1 month ago

(windows11) I was trying to fix my window control buttons being on top of sidebar in fullscreen (F11) (issue) when sidebar is on the same side. So I found this line of code in your file that fixed it :root[sizemode="fullscreen"] #navigator-toolbox { position: relative; }

Now with this code it created this weird bookmarks bar bug where overflow appears and stays when you toggle to Fullscreen (F11) or fullscreen video mode.

I removed all my code and just copy pasted window_control_placeholder_support.css and I reproduced the same thing. image

The problem with that line of code that it conflicts with setting "absolute" on control buttons. Anyway to fix this conflict? The only solution was to not use absolute in fullscreen(F11) but then it doesnt stay at the position you want.


And wanted to ask one more question, what is ID element #window-controls used for? Is it for different operating system, I couldnt find anything in inspect tools.

MrOtherGuy commented 1 month ago

Well, that's definitely weird. I can safely say that I don't have a clue about why that happens, because it can be reproduced with simply this:

:root[sizemode="fullscreen"] #navigator-toolbox { position: relative; }
:root[sizemode="fullscreen"] #TabsToolbar > .titlebar-buttonbox-container:last-child{
  position: absolute;
  display: flex;
  top: 0;
  right:0;
  height: 40px;
}

As soon as .titlebar-buttonbox-container is set to have position absolute, the issue start to happen. I have no idea how the titlebar-buttonbox would affect the overflow button, it doesn't make any sense.

#window-controls however, that's just a leftover from old Firefox versions. Some time back the fullscreen mode had a separate set of titlebar-buttons with that id. I could clean those up because Firefox doesn't seem to use them anywhere anymore.

datguypiko commented 1 month ago

One thing that fixes it is by not using :root[sizemode="fullscreen"] and just setting

#navigator-toolbox { position: relative; }

You think this can cause other issues?

MrOtherGuy commented 1 month ago

Right - that would affect some other styles I'm pretty sure. I decided to go the other way, and simply remove the relative positioning. Instead, styles that do require relative positioning alignment can deal with it themselves - such as hide_tabs_toolbar.css and hide_tabs_with_one_tab_w_window_controls.css - but there might be a few more.