Closed Bug 2059389 Opened 2 months ago Closed 1 month ago

[Nova] Tab Group panels do not match Figma specifications in Dark Theme

Categories

(Firefox :: Tabbed Browser: Tab Groups, defect, P2)

Firefox 155
Desktop
All
defect
Points:
2

Tracking

()

VERIFIED FIXED
156 Branch
Tracking Status
firefox155 --- disabled
firefox156 --- verified
firefox157 --- verified

People

(Reporter: sbadau, Assigned: mhynson)

References

(Blocks 2 open bugs)

Details

(Whiteboard: [fidefe-nova])

Attachments

(2 files)

Found in

  • Nightly 155.0a1

Affected versions

  • Nightly 155.0a1

Tested platforms

  • Affected platforms: Windows 11 x64, macOS 26.04, Ubuntu 24.04

Preconditions

  • Nova is enabled
  • Firefox Dark Theme is enabled

Steps to reproduce

  1. Navigate to any website.
  2. Right-click on the tab -> Add Tab to group -> New Group
  3. Observe the Create Tab Group panel.

Expected result

  • The Create Tab Group panel should match the Figma specifications.

Actual result

  • The Cancel button has the same background as the panel instead of being Gray.
  • The Input field is gray instead of being darker, as per Figma.
  • The panel is narrower in Nightly than in the Figma design, making it appear cut, especially in the upper-left area near the Create Tab Group heading.

Regression range

  • This is not a regression. The current behavior was introduced with the new Nova styling.

Additional notes

  • The same issues can also be observed in the Manage Tab Group panel.
Priority: -- → P2
Severity: -- → S3
  1. The "Create Tab Group" panel state "Cancel" button is type="muted" in the UX spec but type="default" in the codebase
  2. The tab group name input is a <moz-text-input> in the UX spec but <input> in the codebase.
  3. UX spec uses 16px effective padding around the whole panel but the codebase uses 8px effective padding. The codebase uses a combination of the .panel-no-padding class to set 0 padding on the panel and the --panel-menuitem-margin on each item in the panel to get 8px inline margins. Removing .panel-no-padding introduces 16px padding on the panel itself that combines with the 8px inline margins to produce 24px effective padding, which is too much padding.

I also noticed the following:
4. UX spec specifies large (12px) border radius on the panel, but the default --panel-border-radius we're using in the codebase for the panel now is 24px.

Some of these changes would be simple. Not sure if using <moz-text-input> will be a straight drop-in replacement in practice, but it may be worth trying it. Panel padding is kind of tricky because there are some specific configurations that make things easy, but diverging from those configurations makes things harder. :Itiel is a good resource on panels.

Points: --- → 2
Assignee: nobody → mhynson

A few follow ups:

  1. After chatting with UX, the spec has been updated from type="muted" to type="default" for Dark Mode to be consistent with Light Mode.
  2. For the input, the issue is just a visual change. I've simply updated the background color via design token. It turns out that <moz-input-text> is not a straight drop-in replacement. We'd have to ensure parity for accessibility, string, or style changes.
  3. This is addressed in the patch
  4. When chatting with UX, they said we can use the standard panel's border radius of 24px.
  5. I've also slightly bumped the panel width from 22em to 24em for Nova.
Pushed by mhynson@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/f917bf70fd76 https://hg.mozilla.org/integration/autoland/rev/0df5c611d046 Create/Manage Tab Group panel nova tweaks. r=desktop-theme-reviewers,tabbrowser-reviewers,nsharpley,hjones
Status: NEW → RESOLVED
Closed: 1 month ago
Resolution: --- → FIXED
Target Milestone: --- → 156 Branch

The patch landed in nightly and beta is affected.
:mhynson, is this bug important enough to require an uplift?

For more information, please visit BugBot documentation.

Flags: needinfo?(mhynson)
Flags: needinfo?(mhynson)
QA Whiteboard: [S3][qa-found-in-c155][QA-5204] → [S3][qa-found-in-c155][QA-5204] [qa-triage-done-c157/b156] [qa-ver-needed-c157/b156]
Flags: qe-verify+

Verified as fixed using the latest Nightly 157.0a1, Firefox 156.0b4 (with Nova ON) on Windows 11, macOS 26 and Ubuntu 24.04 - the Tab Group panel matches the latest updated Figma specifications and follows the characteristics described in Comment 3.

Status: RESOLVED → VERIFIED
QA Whiteboard: [S3][qa-found-in-c155][QA-5204] [qa-triage-done-c157/b156] [qa-ver-needed-c157/b156] → [S3][qa-found-in-c155][QA-5204] [qa-triage-done-c157/b156] [qa-ver-done-c157/b156]
Flags: qe-verify+
QA Contact: sbadau
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: