Active tab shows gray shade after deactivating High-Contrast theme

NEW
Unassigned

Status

()

Core
ImageLib
P3
normal
a year ago
4 months ago

People

(Reporter: Azmina, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [gfx-noted])

Attachments

(1 attachment)

Comment hidden (obsolete)
(Reporter)

Comment 1

a year ago
Sorry for Messing up the User Story, giving it again below

https://bug1296344.bmoattachments.org/attachment.cgi?id=8782511

User Agent: Mozilla/5.0 (Windows NT 6.3; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/52.0.2743.116 Safari/537.36

Steps to reproduce :
1. Select and activate the High-Contrast theme in Windows.
2. Open Firefox.
3. Deactivate the High-Contrast theme in Windows.

Actual Result :
There is grey shade on two side of the active tab (see the attached screenshot).

Expected Result :
There should not be any grey shade on the active tab.
Moved to a better component, To have more insights. Please change, If there is a better component!
Component: Untriaged → Tabbed Browser

Comment 3

a year ago
I thought this was supposed to be fixed by bug 940625. jwatt is out and seth left. Matt, do you remember how this is supposed to work?
Component: Tabbed Browser → Theme
Depends on: 940625
Flags: needinfo?(MattN+bmo)

Comment 4

a year ago
In general, on Windows, you need to restart Firefox after changing the OS theme, because the Firefox UI is completely "broken".
(In reply to :Gijs Kruitbosch from comment #3)
> I thought this was supposed to be fixed by bug 940625. jwatt is out and seth
> left. Matt, do you remember how this is supposed to work?

I'm not sure what you're asking since the patch in that bug is pretty straightforward to understand.

It seems like finding out whether bug 940625 ever worked on that OS version and if so, getting a regression range would be most useful.

http://mozilla.github.com/mozregression is the tool to use to test old builds.
Has Regression Range: --- → no
Flags: needinfo?(MattN+bmo)
Keywords: qawanted, regressionwindow-wanted
OS: Unspecified → Windows 8.1
Hardware: Unspecified → All
I got this regression range from running MozRegression

Last good revision: 152ef25e89ae (2014-09-10)
First bad revision: 98ea98c8191a (2014-09-11)
Pushlog:
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=152ef25e89ae&tochange=98ea98c8191a
Keywords: regressionwindow-wanted → regression
Status: UNCONFIRMED → NEW
Has Regression Range: no → yes
Ever confirmed: true

Comment 7

a year ago
(In reply to Justin [:JW_SoftvisionQA] from comment #6)
> I got this regression range from running MozRegression
> 
> Last good revision: 152ef25e89ae (2014-09-10)
> First bad revision: 98ea98c8191a (2014-09-11)
> Pushlog:
> https://hg.mozilla.org/mozilla-central/
> pushloghtml?fromchange=152ef25e89ae&tochange=98ea98c8191a

I've looked at this window and I can't for the life of me see any changes that look like they'd be directly related to this behaviour.

Then I tested the builds and afaict they're both broken. I also then checked the build for June 27 and June 26 on Windows and the corners of the tabs are broken there, too. So to the best of my knowledge it just never worked - at least, I don't see this working on Windows 10. I'll try on windows 7 later.
Has Regression Range: yes → irrelevant
Keywords: qawanted, regression

Comment 8

a year ago
(In reply to :Gijs Kruitbosch from comment #7)
> (In reply to Justin [:JW_SoftvisionQA] from comment #6)
> > I got this regression range from running MozRegression
> > 
> > Last good revision: 152ef25e89ae (2014-09-10)
> > First bad revision: 98ea98c8191a (2014-09-11)
> > Pushlog:
> > https://hg.mozilla.org/mozilla-central/
> > pushloghtml?fromchange=152ef25e89ae&tochange=98ea98c8191a
> 
> I've looked at this window and I can't for the life of me see any changes
> that look like they'd be directly related to this behaviour.
> 
> Then I tested the builds and afaict they're both broken. I also then checked
> the build for June 27 and June 26 on Windows and the corners of the tabs are
> broken there, too. So to the best of my knowledge it just never worked - at
> least, I don't see this working on Windows 10. I'll try on windows 7 later.

Doesn't work with the June 26 build on my win7 vm either (switching from aero to classic).

Not sure if this is just us not getting a notification from windows or us not doing enough with the notification, but either way, it doesn't work. I suspect the culprit is still (somehow) imagelib/svg, though, because the colors of the titlebar etc. do change. Though actually, to be fair, content loaded in tabs doesn't get thoroughly updated either, so it could also be a widget/ issue? Up to whoever investigates further. :-\

TBH, not clear that investigating this any further should be a real priority. While I'm sure it's theoretically fixable, a simple restart solves this and the switches are very uncommon in practice.
Component: Theme → ImageLib
Product: Firefox → Core

Updated

a year ago
Whiteboard: [gfx-noted]
Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.