-
-
Notifications
You must be signed in to change notification settings - Fork 139
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
Issue with nav bar width in super theater mode #975
Comments
Hi, sorry about this but can you clarify what you mean? I am having a hard time understanding what you're explaining to me. |
Currently, when I resize the window, the nav bar keeps the same size. So when I resize the window to a smaller width, the nav bar overflows (and when I resize to a larger width, the nav bar doesn't take up the whole width of the window) |
Oh, now I get it, it happens when resizing the window. I will check it out and see what is happening. |
I am not able to replicate the issue, I tried opening a normal video in super theater and resizing the window, the top bar width adjusts correctly, and doing the same for a video with chat, and navigating to a video without chat afterwards. Can you explain which steps I need to follow to be able to trigger the issue on my end? |
Actually, checking your second screenshot shows an odd thing, a scrollbar (I am assuming that's a scrollbar) that is not the normal one: vs the normal scrollbar in the previous screenshot Seems like the window is being resized in a way that is not like my window. Can you take a screenshot of the entire browser window when resized? |
Perfect, thank you for the steps, that was exactly what I needed. I will fix this asap. |
Logic excludes updates when the chat is no longer visible: Iridium/src/firefox/js/background-inject.js Line 993 in dfaa5df
Will have to reset the size to normal behavior when that happens, currently that scenario is not being taken into account. |
Issue should be resolved, seems like during my previous implementations with that feature the reset portion was still using old code. 9d5a655#diff-64234f092fc31df67bfb1b6c9d2ae604e01f21ebc0a378a0a390f359b335e3a4R1046-R1053 I'll be pushing the new version with this fix today. |
The nav bar has a static size in super theater mode.
I'm using Firefox 124 with Iridium 2.0.9.
Example screenshots:
The text was updated successfully, but these errors were encountered: