Closed Bug 1638821 Opened 11 months ago Closed 11 months ago

Make the decision to whether paint focus rings depend on the theme, not the platform

Categories

(Core :: Widget, defect)

defect

Tracking

()

RESOLVED FIXED
mozilla78
Tracking Status
firefox78 --- fixed

People

(Reporter: emilio, Assigned: emilio)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

These ifdefs assume nsNativeTheme to be in use. The decision here should belong to the individual theme, not to generic code.

The current code assumes that nsNativeTheme is in use which breaks with
the non-native theme.

Instead of hackily remove the FOCUS bit, make the theme check for the
FOCUSRING bit, which is the right thing to check for anyway.

Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b8259db1782b
Make the decision to whether paint focus rings depend on the theme, not the platform. r=spohl
Status: NEW → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla78
You need to log in before you can comment on or make changes to this bug.