Open Bug 1734690 Opened 3 years ago Updated 3 years ago

The highlight border of the default button is not displayed with Dark Theme

Categories

(Toolkit :: Themes, defect, P2)

Firefox 95
Desktop
Windows 10
defect

Tracking

()

Tracking Status
firefox-esr78 --- unaffected
firefox-esr91 --- unaffected
firefox93 --- unaffected
firefox94 --- unaffected
firefox95 --- fix-optional

People

(Reporter: alice0775, Unassigned)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: nightly-community, regression)

Attachments

(2 files)

Attached image image.png

Steps To reproduce:

  1. Enable Dark Theme
  2. Open Dialog (not tab modal dialog)
    such as Password Required import From Another Browser

Actual Results:
The highlight border of the default button is not displayed

Regression Window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=ef7b596bb385f46280eb1492009bf8fb541e17db&tochange=b94153cca1f7d677fbbcc1a0d4923d81ddec8958

See Also: → 1734698

This is somewhat expected because the native theme doesn't do anything special about default buttons... I guess the native theme does this, which is just drawing them as focused.

Set release status flags based on info from the regressing bug 1734359

I think bug 1734297 was the regressing bug from that push log.

Blocks: dark-mode

The severity field is not set for this bug.
:dao, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(dao+bmo)
Component: Theme → Widget: Win32
Flags: needinfo?(dao+bmo)
Product: Firefox → Core

The severity field is not set for this bug.
:jimm, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(jmathies)

Windows10 OS Light-mode + Firefox dark mode theme
Regressed by: Bug 1734297

Windows10 OS dark-mode + Firefox system default theme
Regressed by: Bug 1733569

Regressed by: 1734297
No longer regressed by: 1734359
Depends on: 1741359

Maybe this is another reason why common.css should be loaded in dialogs. I did basically that in my custom theme and it looks better imo

Severity: -- → S3
Component: Widget: Win32 → Themes
Flags: needinfo?(jmathies)
Priority: -- → P2
Product: Core → Toolkit
Regressed by: 1733569
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: