horst3180 / arc-firefox-theme

Arc Firefox Theme
Mozilla Public License 2.0
939 stars 48 forks source link

menu presentation broken on Nightly #74

Closed kubrickfr closed 7 years ago

kubrickfr commented 8 years ago

With the Arc theme only, the menu (from the hamburger menu icon) has been completely broken for a week (and I can't screenshot it because it disappears when I press the key). Using the latest git rev.

gyaru commented 8 years ago

You mean like this? (tried without extensions too) example

I'm having the same issue, Firefox 50.0a1. Not really unexpected when using nightly but figured I'd at least report.

kubrickfr commented 8 years ago

Very similar indeed!

p-boiko commented 8 years ago

Confirm broken menu: http://screencloud.net/v/kW1z (and items aren't clickable) The default theme for the contrast: http://screencloud.net/v/7fg7 My Nightly version: http://screencloud.net/v/7Lqk

moriel5 commented 8 years ago

Same here. Started on 50 towards the end, continues on 51. I have to use the ordinary theme now.

wolfland commented 8 years ago

Me Too.. Worked fine with 50 but when Nightly upgraded to 51 yesterday the problem started. Works with both the stock theme and developer mode but fails with Arc theme. screenshot-firefox51-noarc screenshot-firefox51-arctheme

trinaldi commented 8 years ago

Same with Developer Edition (Aurora Channel)

moriel5 commented 8 years ago

It appears that this isn`t limited to Arc, but rather something global. I had tried it with several complete themes, all have the same issues. Also, this is not limited to Linux, I have run into the same issue on Windows 10.

p-boiko commented 8 years ago

More issues on Nightly. Since 26.08.2016 version active tab is not highlighted. I mean active tab background color is the same as inactive.

stonehellway commented 8 years ago

Same issue. I use the same Firefox version. Every Firefox theme is impacted. I use the theme version 48.20160802. Tried a fresh install of Firefox Developer edition: Same issue on Fedora 24 (xfce) and Windows 10.

firefox developer

moriel5 commented 7 years ago

Still there on Nightly 52. As well as active tab not being highlighted, and container tabs menu broken, and downloads quick-menu looking weird (but not broken). screenshot from 2016-09-20 14-00-27

Currently using theme version 49.20160919.

ghost commented 7 years ago

Looks like Firefox beta v50 has updated past the change that causes this.

tiagorinaldi commented 7 years ago

Still facing this issue with Dev. Edition.

raku-cat commented 7 years ago

I switched from aurora to beta because of this, and now beta has updated to a point that it does this. firefox --version just prints 50.0, thats about as far as I know about the specific version I'm on, I use this https://aur.archlinux.org/packages/firefox-beta-bin/

horst3180 commented 7 years ago

Okay, I updated the theme for Firefox 50 beta (firefox-50-beta branch). So this should be fixed at least in the beta version. Note that Aurora and Nightly are moving targets and I simply can't support them properly.

moriel5 commented 7 years ago

It's fine, we know. The whole point (at least for me), aside from using the theme, is to post the possible issues ahead of time, so that you will know what to fix before the changes break the themes for the stable users.

Update: The fix did not work on 52. The workaround for the tabs is working on 52.

tostiheld commented 7 years ago

I had the same issue but for me the fix worked

edit: @moriel5 this is funny, for me it's just the other way around: menu is fixed, tab highlighting is still broken

p-boiko commented 7 years ago

Thanks @moriel5 Surely Nightly IS a moving target :) I just looked a bit through the code, and could the hard selectors be the culprit? I mean like element > element > element... Unfortunately didn't find a time to play with the selectors – the code is huge. I think there is just a bit restructured DOM tree, but element's names should retain. By the way, can You suggest some tool to inspect the Firefox DOM tree? I tried some Inspector, suggested on a Firefox site about themes, but it doesn't seem to work with Nightly's Chrome DOM, only with web-pages, which is already built-in

horst3180 commented 7 years ago

You can type chrome://browser/content/browser.xul into the adress bar and inspect it with the built in inspector (ctrl + shift +c). It doesn't work for the entire UI, but it's something at least.

I usually update the theme by diffing it against the unmodified upstream theme, though.

There's a fix for the recent tab breakage (https://github.com/horst3180/arc-firefox-theme/issues/85), but I havent gotten around to implementing it.

wolfland commented 7 years ago

The beta works for me on Nightly (52) as well.

Thanks

On 10/02/2016 04:05 PM, thomas wrote:

I had the same issue but for me the fix worked

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/horst3180/arc-firefox-theme/issues/74#issuecomment-250992315, or mute the thread https://github.com/notifications/unsubscribe-auth/AIjIgDyygL6uiArLeTCvHXVcgEWWrLLcks5qwA5wgaJpZM4JT2sI.

p-boiko commented 7 years ago

Thank you @horst3180 , very useful tips :)

moriel5 commented 7 years ago

Strange, since for me it didn't work on either Arch (Antergos to be precise, however with Apricity, VeltOS, and several other repos added as well) nor Solus.

And I use the upstream (as in vanilla, binaries built by Mozilla themeselves) Nightly, since I download it directly from Mozilla.

awalgarg commented 7 years ago

@p-boiko

By the way, can You suggest some tool to inspect the Firefox DOM tree?

@horst3180

You can type chrome://browser/content/browser.xul into the adress bar and inspect it with the built in inspector (ctrl + shift +c). It doesn't work for the entire UI, but it's something at least.

For "the entire UI", you can enable browser debugging by opening devtools (F12) on any page, going to settings, and checking "Enable browser chrome and add-on debugging toolboxes" and "Enable remote debugging", and then pressing Ctrl+Alt+Shift+I. See https://developer.mozilla.org/en-US/docs/Tools/Browser_Toolbox for more info.

bpseudopod commented 7 years ago

Still a problem as of 2016-10-27 on Dev Edition. firefox-developer --version reports Mozilla Firefox 51.0a2.

peronero commented 7 years ago

Can confirm the beta branch fixes this in 52.0a1 (2016-11-07).

qazip commented 7 years ago

I'm using Firefox beta (50) in Linux and Arc theme (Arc theme 49.20160920) it's bugged here. Even add-ons that open windows are not responding (umatrix).

firefox_arc_theme_bug

horst3180 commented 7 years ago

@mimf 49.20160920 is for Firefox 49. Build the theme from the firefox-50-beta branch to get the fix, or wait for the Firefox 50 release (tomorrow).