Closed Bug 1690186 Opened 3 years ago Closed 3 years ago

Proton Tab Styling Conflicts with Container Tabs Indicator

Categories

(Firefox :: Tabbed Browser, defect, P1)

Firefox 87
defect

Tracking

()

VERIFIED FIXED
88 Branch
Tracking Status
firefox88 --- verified

People

(Reporter: bobsfree, Assigned: jaws)

References

(Blocks 1 open bug)

Details

(Whiteboard: [proton-tabs-bar])

Attachments

(5 files)

Attached file about:support

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:87.0) Gecko/20100101 Firefox/87.0

Steps to reproduce:

Enabled Container Tabs through one of the many different addons.

Actual results:

The Proton Tab subtext has insufficient padding when the coloured container tabs indicator is enabled.

Expected results:

Sufficient padding be present between the media status subtext and the container tabs indicated

Please see the attached screenshot to see the issue.

OS: Unspecified → All
Hardware: Unspecified → All
Component: Untriaged → Tabbed Browser
Summary: Proton Tab Styling Conflicts with Container Tabs → Proton Tab Styling Conflicts with Container Tabs Indicator
Whiteboard: [proton-tabs-bar]

I don't want to spam this bug, but I just wanted to leave a quick note that this is still valid after the fixing of Bug 1688072, and Bug 1689852.

It is much improved, but a slight lack of padding at the bottom while media is playing still arguably looks a little unintentional, as can be seen in the newly attached screenshot.

The amount of impact obviously also seems to depend on the name of the page, particularly if the page title contains a g, j, p, q or y, per the last new screenshot.

Aaron, can you please help provide details here?

Flags: needinfo?(abenson)
Assignee: nobody → jaws
Severity: -- → S3
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Priority: -- → P1
Flags: needinfo?(abenson) → needinfo?(kcaldwell)
Pushed by jwein@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/62b9712b6c1e
Adjust the container tab styling to fit with the Proton theme. r=mconley
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 88 Branch

UX Update: Figma file updated with new visual spec for container visual treatment. Jaws has link and details.

Flags: needinfo?(kcaldwell)

Hello I have managed to reproduce the issue with Fx 87.0a1 (2021-02-01) on Ubuntu 20, I can confirm that the issue is fixed with Fx 88.0a1(2021-03-16). I will update the flags accordingly.

Thank you!

Status: RESOLVED → VERIFIED
Regressions: 1701479
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: