Closed Bug 1646878 Opened 4 years ago Closed 4 years ago

Ship stylesheets for browser_style

Categories

(Thunderbird :: Add-Ons: Extensions API, enhancement)

enhancement

Tracking

(thunderbird78 fixed)

RESOLVED FIXED
Thunderbird 79.0
Tracking Status
thunderbird78 --- fixed

People

(Reporter: darktrojan, Assigned: darktrojan)

Details

Attachments

(1 file)

Browser actions and the other actions have an option to load a browser stylesheet so that the UI looks more similar to the application UI. We've never shipped these stylesheets or made the option work, but we should. Also at least one of them is used in the add-on manager preferences.

I picked up a few minor updates to ExtensionPopups.jsm while I was there.

Attachment #9157818 - Flags: review?(mkmelin+mozilla)
Comment on attachment 9157818 [details] [diff] [review]
1646878-browser-style-1.diff

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

LGTM, r=mkmelin
Attachment #9157818 - Flags: review?(mkmelin+mozilla) → review+

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/29838b8b530f
Ship stylesheets for browser_style. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 79.0

I guess we should uplift?

Comment on attachment 9157818 [details] [diff] [review]
1646878-browser-style-1.diff

(In reply to Magnus Melin [:mkmelin] from comment #4)
> I guess we should uplift?

Yeah, there's no harm. I'd push it into a point release anyway, so it might as well be point-0.
Attachment #9157818 - Flags: approval-comm-beta?
Comment on attachment 9157818 [details] [diff] [review]
1646878-browser-style-1.diff

Also no regressions filed since landing, so Approved for beta
Attachment #9157818 - Flags: approval-comm-beta? → approval-comm-beta+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: