The default bug view has changed. See this FAQ.

Use the global close icon for Tabs

RESOLVED FIXED in Thunderbird 43.0

Status

Thunderbird
Theme
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: Paenglab, Assigned: Paenglab)

Tracking

unspecified
Thunderbird 43.0

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
FX is using the close icons from global/icons. When we switch to this icons for the tabs we get automatically the icons FX is using and we have then also the HiDPI versions automatically.

Referenced bugs:
Bug 1168528 - Tab close icons on Windows HiDPI are blurry
Bug 1173729 - Update generic close icon and new tab button icon on Windows 10
(Assignee)

Comment 1

2 years ago
Created attachment 8646864 [details] [diff] [review]
globalIcon.patch

I've built and tested this patch on Linux and Windows. On OS X I patched the omni.ja to test.

Philipp, if you can build and test this on OS X then we can be sure it works everywhere.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8646864 - Flags: review?(philipp)
Comment on attachment 8646864 [details] [diff] [review]
globalIcon.patch

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

Code looks fine, so r+ for now. My builds are messed up at the moment, I'd have to clobber to test this. Maybe you can start a try build instead?
Attachment #8646864 - Flags: review?(philipp) → review+
(Assignee)

Comment 3

2 years ago
The Nightlies aren't also not building, but this seems to be a infra issue.

Try: https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=1baa4272df6d
(Assignee)

Comment 4

2 years ago
(In reply to Richard Marti (:Paenglab) from comment #3)
> The Nightlies aren't also not building, but this seems to be a infra issue.
> 
> Try:
> https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=1baa4272df6d

The try build failed too on OS X.
(Assignee)

Updated

2 years ago
Keywords: checkin-needed

Comment 5

2 years ago
https://hg.mozilla.org/comm-central/rev/1d40ee3eb12d433ef53e5052147186aab53555f9
Bug 1193690 - Use the global close icon for Tabs. r=Fallen

Updated

2 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 43.0
You need to log in before you can comment on or make changes to this bug.