Closed Bug 1703029 Opened 4 years ago Closed 1 year ago

Container tab indicator barely visible with certain themes

Categories

(Firefox :: Theme, defect, P2)

Firefox 89
defect

Tracking

()

RESOLVED DUPLICATE of bug 1704201
Tracking Status
firefox89 --- affected

People

(Reporter: belfox_ruben, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [proton-tabs-bar] [priority:2c])

Attachments

(3 files)

Steps to reproduce:

  1. Open Firefox Nightly 89.0a1.
  2. Activate the Alpenglow theme.
  3. Configure a new container "Google", using red as the container color.
  4. Open a new container tab in this "Google" container.

Expected result:
The tab, whether focused or not, is recognizable as a container tab.

Actual result:
The tab, whether focused or not, is is not recognizable as a container tab.

Attached image Solution_proposal1.png

First proposal that was discussed in the Matrix "Proton" channel on April 5th.
Mock-up by non-UX people, so review necessary.

Attachment #9213581 - Flags: ui-review+
Attached image Solution_proposal2.png

Second proposal that was discussed in the Matrix "Proton" channel on April 5th.
Mock-up by non-UX people, so review necessary.

Attachment #9213582 - Flags: ui-review+
Whiteboard: [proton-tabs-bar]
Status: UNCONFIRMED → NEW
Ever confirmed: true
Priority: -- → P2
Whiteboard: [proton-tabs-bar] → [proton-tabs-bar] [priority:2c]

Moving the indicator inside the tab content isn't going to solve the issue for all themes. A better solution might be to allow custom colors configurable by the user (bug 1325057) and/or through the Theming API.

Flags: needinfo?(belfox_ruben)
See Also: → 1325057

(In reply to Danny Colin [:sdk] from comment #3)

Moving the indicator inside the tab content isn't going to solve the issue for all themes. A better solution might be to allow custom colors configurable by the user (bug 1325057) and/or through the Theming API.

Fair enough!

Flags: needinfo?(belfox_ruben)
Status: NEW → RESOLVED
Closed: 1 year ago
Duplicate of bug: 1704201
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: