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

[Firefox 65+] 'tabs below' / 'tabs not on top' changes #164

Closed
Jgr9 opened this issue Nov 7, 2018 · 212 comments
Closed

[Firefox 65+] 'tabs below' / 'tabs not on top' changes #164

Jgr9 opened this issue Nov 7, 2018 · 212 comments

Comments

@Jgr9
Copy link

Jgr9 commented Nov 7, 2018

Ok, so I'm here (for the first time) after my Tabs Below the Nav Bar and Bookmark Bar userChrome code has apparently been broken again by the most recent Nightly. 20181106

I've tried yours, and if I've implemented it correctly (I'm kind of a novice), it also doesn't seem to have an effect. I haven't used a css folder and @import before, so I created one in the folder along with my userChrome and put your extra css in it. I uncommented the line in your userChrome, and also just tried the line all by itself. Like the code I have, it doesn't seem to be working anymore on the new Nightly, unless I did something wrong or there is something else that I need to do.

65.0a1 (2018-11-06) - 20181106 - Windows 64 bit

Oh and I'm just realizing now you're the CTR guy... lol

@Aris-t2
Copy link
Owner

Aris-t2 commented Nov 7, 2018

You have done nothing wrong. Latest update introduced some heavy changes to the top content, which broke some things.

The "titlebar box" is now part of the "navigator-toolbox" and contains menubar and tabs toolbar. This means we can not move tabs toolbar below navigation/bookmarks toolbar without moving menubar too. I have to take a deeper look at it.

@Jgr9
Copy link
Author

Jgr9 commented Nov 7, 2018

Well that doesn't sound good if things can't be created with css. Because I still use the old menubar. And nav and bookmark bar as well of course. But if they can't be re-ordered anyways now...

@Aris-t2
Copy link
Owner

Aris-t2 commented Nov 7, 2018

There are still ways to achieve this in CSS, but things got more complicated and require more code adjustments. Wait for an update.

@Aris-t2 Aris-t2 added the BUG label Nov 7, 2018
@Aris-t2 Aris-t2 self-assigned this Nov 7, 2018
@Aris-t2
Copy link
Owner

Aris-t2 commented Nov 7, 2018

@Aris-t2 Aris-t2 closed this as completed Nov 7, 2018
@drJeckyll
Copy link

on my 65.0a1 (2018-11-07) (64-bit) on Linux Tabs stay on top
put:
@import "./css/tabs/tabs_below_navigation_toolbar.css";
@import "./css/tabs/tabs_below_navigation_toolbar_fx65.css";
together and only _fx65 nothing changes

@Aris-t2 Aris-t2 reopened this Nov 7, 2018
@Aris-t2 Aris-t2 changed the title Tabs Below Might Have Been Broken Again. [Linux][Fx65] 'tabs below' issue (fixed on Windows with v2.2.3 release) Nov 7, 2018
@Aris-t2
Copy link
Owner

Aris-t2 commented Nov 7, 2018

I have to check this.

Current tests were only run on Windows 7&10.

@Aris-t2
Copy link
Owner

Aris-t2 commented Nov 7, 2018

I've tested this on Ubuntu and it works. Only menubar needs some visual adjustments.

With custom_css_for_fx_v2.2.3.1_test1.zip

123

@drJeckyll
Copy link

That's weird. Have to move @import clause on top of the file before my other custom rules to move tabs on bottom. It almost works. Menu is bit lower, and go over location. I guess I can play with values here. (for some reason can't attach image here).

@Aris-t2
Copy link
Owner

Aris-t2 commented Nov 7, 2018

Drag and drop a jpg or png file into text field. This attaches images to the post.

I don't know about your custom css tweaks, but you always have to test the whole package I provide, so we can compare results.

@drJeckyll
Copy link

It's the whole package, I just have some additional rules, but nothing about tabs.

screenshot at 2018-11-07 18-46-27
(attachments now work)

@nicolaasjan
Copy link

Hi,
Here on Linux Mint 17.3 and 'custom_css_for_fx_v2.2.3.1_test2' it looks like this:

screenshot nightly

Not quite right it seems...

@BrandtnerKPW
Copy link

BrandtnerKPW commented Nov 7, 2018

On Windows 10 with v2.2.3 while watching a video in fullscreen mode, the minimize, maximize and close buttons of the browser are displayed in the upper right corner of the screen.
default
Firefox Nightly 65.0a1 (2018-11-07) (64-bit)

@Jgr9
Copy link
Author

Jgr9 commented Nov 7, 2018

Menubar issue (Windows 10). Other things seem to be ok. (Except for Brandtner's issue as well. ^)

menubarissue

@Aris-t2
Copy link
Owner

Aris-t2 commented Nov 7, 2018

@ALL
Test custom_css_for_fx_v2.2.3.1_test3.zip

@drJeckyll
Test default empty menubar.

@nicolaasjan
Test custom_css_for_fx_v2.2.3.1_test3.zip

@BrandtnerKPW
Test custom_css_for_fx_v2.2.3.1_test3.zip

@Jgr9
I need more info.
Looks like this for me (Win10+lw-theme+tabs-not-on-top(65))
1
e:

@Jgr9
Copy link
Author

Jgr9 commented Nov 7, 2018

Um, I'm not really sure what you need really. Windows 10 scaled at 125% 1680x1050

I might have some old Firefox config setting that I set a long time ago for "1:1" scaling, but I don't remember.

Otherwise, all I have in my userChrome and css is yours, not any of mine.

Edit: Oh, nevermind.... I broke the "one at a time" rule. For some reason I didn't see it in that section and thought multiple would be needed (though.... expected not to need the Mac ones).

@drJeckyll
Copy link

with test3 it looks OK - just need to remove that ugly line under menu
screenshot at 2018-11-07 23-00-49

@BrandtnerKPW
Copy link

My problem is solved. Thank you!

@Jgr9
Copy link
Author

Jgr9 commented Nov 7, 2018

Hmm, really? I fixed my issue, but I'm still getting the min/max & Close buttons on my Fullscreen with test3 What specific file is that related to?

Edit: Oh, nvm. Not video, just F11 it's still there. So is my scrollbar, but in reality, those might not be such bad things really, unless I really wanted something fullscreen with nothing else.

@nicolaasjan
Copy link

This last test build (custom_css_for_fx_v2.2.3.1_test3) makes it look much better:

screenshot nightly 2

Thanks!

@Aris-t2
Copy link
Owner

Aris-t2 commented Nov 8, 2018

@drJeckyll
Could you experiment with this code and tell me, if the border is gone?

#nav-bar,
#navigator-toolbox {
  border-top: 0 !important;
  box-shadow: unset !important;
}

@Jgr9
The fullscreen caption buttons will be gone too on next upate.

@nicolaasjan @BrandtnerKPW
Good to hear.

Sadly different Linux distributions use different height values for menubar making universal adjustments impossible.

@nicolaasjan
Copy link

< --- >
snip
< --- >

@nicolaasjan @BrandtnerKPW
Good to hear.

Sadly different Linux distributions use different height values for menubar making universal adjustments impossible.

Ah, I see.

What can be changed in 'tabs_below_navigation_toolbar_fx65.css' when something like this happens again?

@drJeckyll
Copy link

@Aris-t2 That piece works. Thank you.
Any idea how to remove line bellow tabs?
screenshot at 2018-11-08 13-18-51

@Aris-t2
Copy link
Owner

Aris-t2 commented Nov 8, 2018

@nicolaasjan
The are no guarantees with Mozilla whether something like this is happening again or not. No way to know how of if this will be changeable then in the future.

@drJeckyll
This works on Windows (not tested on Linux yet):

#TabsToolbar {
  border-bottom: unset !important;
} 

@drJeckyll
Copy link

@Aris-t2 that works too. Thanks again

@Aris-t2 Aris-t2 changed the title [Linux][Fx65] 'tabs below' issue (fixed on Windows with v2.2.3 release) [Linux][Fx65] 'tabs below' issue (fixed) Nov 8, 2018
@Juvencus
Copy link

Juvencus commented Nov 3, 2019

Titlebar option on macOS and Linux is unstable and should not be used when customizing user interface with tab changes like multirow tabs.

Multirow tabs clip into the webpage regardless of the titlebar being on or off in Customize

@Aris-t2
Copy link
Owner

Aris-t2 commented Nov 3, 2019

@Lyberta

I don't see what is broken on your end when trying to use the statusbar code. Could you post some screenshots of how it has to look and how it looks with this projects files?

By "tabs below" you mean tabs below navigation bar, right? Because tabs below main content of course breaks any other browser bottom modifications.

@Aris-t2
Copy link
Owner

Aris-t2 commented Nov 3, 2019

@Juvencus

Using this combination on macOS is a different story. After running some tests I found the issue. Next update will offer a new option to be used on macOS.

2.8.2
tabs_below_navigation_toolbar_fx65_v2_macOS.css + tabs_multiple_lines_fx71.css

@Juvencus
Copy link

Juvencus commented Nov 3, 2019

@Juvencus

Using this combination on macOS is a different story. After running some tests I found the issue. Next update will offer a new option to be used on macOS.

2.8.2
tabs_below_navigation_toolbar_fx65_v2_macOS.css + tabs_multiple_lines_fx71.css

As always, amazing. Thanks for your work!

@ghost
Copy link

ghost commented Nov 6, 2019

@Lyberta

I don't see what is broken on your end when trying to use the statusbar code. Could you post some screenshots of how it has to look and how it looks with this projects files?

/* STATUS BAR */

@import "./StatusBar.css";

/* Remove the Hamburger Button */
#PanelUI-menu-button {display: none;}

StatusBar

/* TABS ON BOTTOM */

@import "./css/tabs/tabs_below_navigation_toolbar_fx68.css";

:-moz-any(.tabs-newtab-button,#tabs-newtab-button) .toolbarbutton-icon {
  padding: 4px !important;
  margin: 0px !important;
  width: 24px !important;
  height: 24px !important;
}

/* STATUS BAR */

@import "./StatusBar.css";

/* Remove the Hamburger Button */
#PanelUI-menu-button {display: none;}

StatusBarAndTabs

The content of StatusBar.css is in my previous post.

EDIT: By messing with debugger and searching I've found that all imports must go first so I put fix for new tab button inside tabs_below_navigation_toolbar_fx68.css and it works.

@Aris-t2
Copy link
Owner

Aris-t2 commented Nov 6, 2019

Try to add the code from "statusbar.css" to the end of your userChrome.css for testing purposes, it does the job for me:

1

@ghost
Copy link

ghost commented Nov 7, 2019

Yes, my problem was that I've had a regular CSS rule before the import. CSS forbids that.

@rathord1912
Copy link

It does not work like this.

Try to only use this code without enabling tabs_below_main_content_fx65.css:

#main-window { 
-moz-box-direction: reverse;
}

Hi Aris,

in FF72+ the reverse code, to place all toolbars below the content area, doesn't work anymore. Is there another way?

@Aris-t2
Copy link
Owner

Aris-t2 commented Jan 9, 2020

Try this:

#appMenu-viewCache + box {
  -moz-box-ordinal-group: 1000 !important;
}

@rathord1912
Copy link

Perfect. It's just working fine.

@Aris-t2
Copy link
Owner

Aris-t2 commented Jan 10, 2020

I don't know how long this code will work. Devs like to move things into id-free boxes lately.

@ArkadiuszMichalski
Copy link

ArkadiuszMichalski commented Jan 10, 2020

Last Nightly 74, when using @import "./css/tabs/tabs_below_navigation_toolbar_fx72.css" and Title bar off (via Firefox setting) my button min/back/close in right corner is visible in 1/3 size, look:
Schowek04
In first img when @import "./css/tabs/tabs_below_navigation_toolbar_fx72.css" is on, and in second when off. Only I have this problem?
Problem not exist when I have Title bar on in Customization...

@Aris-t2
Copy link
Owner

Aris-t2 commented Jan 11, 2020

I will look into this.

Problem not exist when I have Title bar on in Customization...

In this case the OS titlebar is used. It does not get styled by themes and therefore does not create such overlapping issues.

@ArkadiuszMichalski
Copy link

Problem make lwt: https://addons.mozilla.org/en-US/firefox/addon/pinstripe-blue/
If off then buttons are visible.

@Aris-t2
Copy link
Owner

Aris-t2 commented Jan 11, 2020

It might have something to do with the following code:

#main-window[tabsintitlebar]:not([inDOMFullscreen="true"]):-moz-lwtheme #toolbar-menubar ~ #TabsToolbar .titlebar-buttonbox-container {
  display: block !important;
  margin-top: -13px !important;
}

I can not troubleshoot on Win 7 atm, try to modify the above value.

@ArkadiuszMichalski
Copy link

Yes, removing margin-top... fix problem, looks like its not necessary now, thx.

@Aris-t2
Copy link
Owner

Aris-t2 commented Jan 11, 2020

Without the code I get this glitch:
image

Leaving the code shows things correctly:
image

@ArkadiuszMichalski
Copy link

Yes, in stable version, but like wrote above I use last Nightly74, don't know if this change will be needed in 73 or 74 (probably in 74 because for long time everything was ok, few days ago this appear).

@Aris-t2
Copy link
Owner

Aris-t2 commented Jan 11, 2020

Overlooked that while testing on W7. ;-)

@Aris-t2
Copy link
Owner

Aris-t2 commented Jan 11, 2020

I think we might get it working for all versions by modifying the code.

Change this:

#main-window[tabsintitlebar]:not([inDOMFullscreen="true"]):-moz-lwtheme #toolbar-menubar ~ #TabsToolbar .titlebar-buttonbox-container {
  display: block !important;
  margin-top: -13px !important;
}

to this:

#main-window[tabsintitlebar]:not([inDOMFullscreen="true"]):-moz-lwtheme #toolbar-menubar ~ #TabsToolbar .titlebar-buttonbox-container {
  display: flex !important;
}

@ArkadiuszMichalski
Copy link

If this one rule also work in FX72/73 then yes, I check flex rule in Nightly 74 and looks good, the same as block.

@ryuukenzenkou
Copy link

ryuukenzenkou commented Jan 25, 2020

hey i want to request some help. i was using firefox 71 and everything was smooth, then updated to v72 and using your v3.0.0 CSS and the nightmare began. broken addon toolbar at bottom, tabs not working as they should, had to go through the code to fix firefox app button and so on...
have a look here...
Captura
i've tried everything that i could to fix these 2 issues, but no luck, so please help... the tab and the addon bar at bottom are the only thing that bug me.. (pun intended).. but if there's a way to fix it, ill be glad to know. btw ill share the CSS code in order to let you inspect it and check whats wrong.
chrome.zip
im not a programmer, but i do know a bit about the stuff as i am currently an student. please help.

edit: ill keep reading and trying to fix it on my own.
edit2: using windows 7 pro x64, firefox v72 x64, default dark theme with aero enabled and no other tab addon installed. was working fine until firefox stable/release v71

@Aris-t2
Copy link
Owner

Aris-t2 commented Jan 25, 2020

You initial screenshot pointed out, that layout.css.xul-box-display-values.survive-blockification.enabled was set to false on your system. This causes major issues even on default configuration without any modifications. Verify that pref is set to true inside about:config.

@fbnk
Copy link

fbnk commented Jan 25, 2020

Hi,
is there any setting to get a bit more space between tabs and the fav bar? Plus there is a little space below the tabs, like they are "floating".

Bildschirmfoto 2020-01-25 um 09 43 11

At the moment, this is the only tab related setting which is imported:
@import "./css/tabs/tabs_below_navigation_toolbar_fx65_macOS_default_tabs.css"; /**/

Thanks!

FF 72.0.2
macOS 10.15
CustomCSSforFx 3.0.0

@Aris-t2
Copy link
Owner

Aris-t2 commented Jan 25, 2020

Please use tabs_below_navigation_toolbar_fx65_macOS.css as workaround. Not sure what exactly changed on macOS atm, but it seems like tabs_below_navigation_toolbar_fx65_macOS_default_tabs.css might not be required anymore.

This code will increase the space between bookmarks toolbar and tabs toolbar in your config:

#navigator-toolbox #personal-bookmarks {
  padding-bottom: 20px !important;
}

@fbnk
Copy link

fbnk commented Jan 25, 2020

Thx for the quick reply! I guess the space is just fine now, after the switch :)
And thx for the continuous development as well!

@ryuukenzenkou
Copy link

@Aris-t2 I'll change the values as you mentioned and see what happens. I'll reply soon as possible.

@Aris-t2
Copy link
Owner

Aris-t2 commented Jul 20, 2021

I think we can close this one now.
Fx 65 release was a long time ago ;-)

@Aris-t2 Aris-t2 closed this as completed Jul 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests