[CSD] Titlebar does not visually distinguish between active and inactive windows

VERIFIED FIXED in Firefox 65

Status

()

defect
VERIFIED FIXED
Last year
7 months ago

People

(Reporter: blokowski420, Assigned: stransky)

Tracking

(Blocks 1 bug)

60 Branch
mozilla64
Unspecified
Linux
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox64 wontfix, firefox65 verified)

Details

Attachments

(4 attachments, 1 obsolete attachment)

Reporter

Description

Last year
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Firefox/60.0
Build ID: 20180301024724

Steps to reproduce:

1. Open Firefox
2. Go to about:config and set browser.tabs.drawInTitlebar to 'true'
3. Open a different application (for example Nautilus or a Terminal)
4. Switch back and forth between the two applications (alt+tab)


Actual results:

The Firefox titlebar doesn't change visually, regardless of whether the window is active or inactive.


Expected results:

With many desktop environments and themes there is a visual indication, when a window is inactive.
For example, in GNOME with the Arc GTK theme the window control buttons switch to grayscale and the overall titlebar content is muted (see attachment).
Reporter

Updated

Last year
Blocks: gtktitlebar
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Unspecified → Linux
Assignee

Comment 1

10 months ago
This is partially caused by wrong titlebar style which is solved at Bug 1408360 and also the style is animated which we don't support for the titlebar element.
Assignee: nobody → stransky
Assignee

Updated

10 months ago
Depends on: 1408360
Assignee

Updated

10 months ago
Depends on: 1489097
Assignee

Updated

10 months ago
Blocks: 1489963
Assignee

Comment 2

10 months ago
Let's fix headerbar in this bug and buttons at Bug 1489963.
Summary: [CSD] Titlebar and window control buttons do not visually distinguish between active and inactive windows → [CSD] Titlebar does not visually distinguish between active and inactive windows
Assignee

Updated

10 months ago
No longer depends on: 1489097

Comment 4

10 months ago
Comment on attachment 9007740 [details]
Bug 1442755 - Redraw titlebar widgets when application focus changes and draw inactive titlebar with GTK_STATE_FLAG_BACKDROP state, r=jhorak

Jan Horak [:jhorak] has approved the revision.
Attachment #9007740 - Flags: review+
Assignee

Updated

10 months ago
Keywords: checkin-needed

Comment 5

10 months ago
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/19e2c611e889
Redraw titlebar widgets when application focus changes and draw inactive titlebar with GTK_STATE_FLAG_BACKDROP state, r=jhorak
Keywords: checkin-needed

Comment 11

10 months ago
Comment on attachment 9008180 [details]
Bug 1442755 - Redraw titlebar widgets when application focus changes and draw inactive titlebar with GTK_STATE_FLAG_BACKDROP state, r=jhorak

Jan Horak [:jhorak] has approved the revision.
Attachment #9008180 - Flags: review+
Assignee

Updated

10 months ago
Attachment #9007740 - Attachment is obsolete: true
Flags: needinfo?(stransky)
Assignee

Updated

10 months ago
Keywords: checkin-needed

Comment 12

10 months ago
Pushed by ebalazs@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/511150b11de9
Redraw titlebar widgets when application focus changes and draw inactive titlebar with GTK_STATE_FLAG_BACKDROP state, r=jhorak
Keywords: checkin-needed

Comment 13

10 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/511150b11de9
Status: NEW → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
Assignee

Updated

9 months ago
Blocks: 1491808

Updated

9 months ago
Depends on: 1497534
Assignee

Updated

8 months ago
Depends on: 1506050
Assignee

Comment 14

8 months ago
We may disable this at Beta (Firefox 64) as it comes with a visual regressions (Bug 1506050 and Bug 1497534).
Would you mind providing a backout patch for beta?
Flags: needinfo?(stransky)
Assignee

Comment 16

8 months ago
(In reply to Julien Cristau [:jcristau] from comment #15)
> Would you mind providing a backout patch for beta?

Sure, I'll do that. Keeping needinfo to remind me.
Assignee

Comment 17

7 months ago
Flags: needinfo?(stransky)
Attachment #9024657 - Flags: review?(jhorak)
Comment on attachment 9024657 [details] [diff] [review]
disable titlebar theming for Firefox 64 (Beta)

Review of attachment 9024657 [details] [diff] [review]:
-----------------------------------------------------------------

Please comment also in nsNativeThemeGTK::WidgetStateChanged, r+ with that fixed.
Attachment #9024657 - Flags: review?(jhorak) → review+
Assignee

Comment 19

7 months ago
Updated one, thanks.
Assignee

Comment 20

7 months ago
Comment on attachment 9025562 [details] [diff] [review]
patch for check-in

[Beta/Release Uplift Approval Request]

Feature/Bug causing the regression: Bug 1497534

User impact if declined: Back out this Bug for beta as it may cause titlebar appearance regressions (Bug 1497534).

Is this code covered by automated tests?: No

Has the fix been verified in Nightly?: No

Needs manual test from QE?: Yes

If yes, steps to reproduce: 1) disable system titlebar (Customize -> Titlebar checkbox)
2) open another application (terminal for instance)
3) switch to it by alt+tab or so and check that firefix titlebar still has active appearance and it's not changed when switching to another apps.

List of other uplifts needed: None

Risk to taking this patch: Low

Why is the change risky/not risky? (and alternatives if risky): Back out of a fix, Linux only.

String changes made/needed:
Attachment #9025562 - Flags: approval-mozilla-beta?
Comment on attachment 9025562 [details] [diff] [review]
patch for check-in

Thanks Martin, approved for 64.0b11
Attachment #9025562 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Whiteboard: [checkin-needed-beta]
Whiteboard: [checkin-needed-beta]
I have managed to reproduce the issue using Fx60.0a1 buildID: 20180301223350 on Ubuntu 16.04.

The issue is verified fixed using Fx65.0b2 and Fx65.0a1 on Ubuntu 16.04. The Firefox window is now correctly set as inactive when it is not the main focus (buttons become greyed out). This was tested using Ambience default theme on Ubuntu.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.