Closed Bug 1033570 Opened 7 years ago Closed 7 years ago

In-content dialogs close-icon too wide with HiDPI

Categories

(Toolkit :: Themes, defect)

All
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla33

People

(Reporter: Paenglab, Assigned: Paenglab)

References

Details

Attachments

(1 file)

The in-content dialog close-icon is to wide with HiDPI.
The close-icon button used on in-content dialogs has a button-box in his structure. This patch adds a selector to also fit for the width: 16px.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8449651 - Flags: review?(dolske)
Blocks: 1021618
Comment on attachment 8449651 [details] [diff] [review]
close-iconFix.patch

Review of attachment 8449651 [details] [diff] [review]:
-----------------------------------------------------------------

r+ assuming you tested it and it fixes the problem, as I didn't. :)
Attachment #8449651 - Flags: review?(dolske) → review+
Keywords: checkin-needed
Hi,

could you provide a Try link. Suggestions for what to run if you haven't
yet can be found here:
https://wiki.mozilla.org/Sheriffing/How:To:Recommended_Try_Practices
Isn't it to much to let go through the try building for a one line CSS patch?

But I'll start one.
(In reply to Richard Marti (:Paenglab) from comment #4)
> Isn't it to much to let go through the try building for a one line CSS patch?

Indeed. Tomcat, try to use your judgement in such situations :)
https://hg.mozilla.org/mozilla-central/rev/043995106767
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → mozilla33
Summary: In-content dialogs close-icon to wide with HiDPI → In-content dialogs close-icon too wide with HiDPI
You need to log in before you can comment on or make changes to this bug.