Closed
Bug 1593224
Opened 1 year ago
Closed 1 year ago
Port Bug 1516091 to Thunderbird (remove sidebarheader element, convert to <box class="sidebar-header">)
Categories
(Thunderbird :: General, task, P1)
Thunderbird
General
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 72.0
People
(Reporter: mkmelin, Assigned: mkmelin)
References
Details
Attachments
(1 file, 1 obsolete file)
17.00 KB,
patch
|
mkmelin
:
review+
|
Details | Diff | Splinter Review |
Bug 1516091 is getting rid of <sidebarheader>
Per Tim's comment: "I guess they could be changed to box class="sidebar-header" and the styling could be ported over."
Assignee | ||
Updated•1 year ago
|
Priority: -- → P1
Assignee | ||
Updated•1 year ago
|
Summary: Port Bug 1516091 to Thunderbird (convert/remove sidebarheader element) → Port Bug 1516091 to Thunderbird (remove sidebarheader element, convert to <box class="sidebar-header">)
Assignee | ||
Comment 1•1 year ago
|
||
Attachment #9105764 -
Flags: review?(richard.marti)
Assignee | ||
Updated•1 year ago
|
Status: NEW → ASSIGNED
Comment 2•1 year ago
|
||
Comment on attachment 9105764 [details] [diff] [review] bug1593224_sidebarheader.patch Review of attachment 9105764 [details] [diff] [review]: ----------------------------------------------------------------- Thanks, r+ with the comments considered. Patch can land without waiting for the M-C bug. ::: mail/base/content/messenger.xul @@ +494,5 @@ > #include mainMailToolbox.inc.xul > > <box id="messengerBox" orient="horizontal" flex="1" minheight="100" height="100" persist="height"> > <vbox id="folderPaneBox" minwidth="125" width="200" persist="width"> > + <box id="folderPaneHeader" class="sidebar-header" hidden="true" align="center"/> Shouldn't we remove this header with the CSS rules? It seem never used. ::: mail/themes/shared/mail/messengercompose.css @@ +155,5 @@ > list-style-image: var(--webextension-menuitem-image-2x, inherit) !important; > } > } > > +.sidebar-header { This and the following rule aren't needed because the ones from messenger.css are already used.
Attachment #9105764 -
Flags: review?(richard.marti) → review+
Assignee | ||
Comment 3•1 year ago
|
||
(In reply to Richard Marti (:Paenglab) from comment #2)
Shouldn't we remove this header with the CSS rules? It seem never used.
Yeah let's remove it.
This and the following rule aren't needed because the ones from
messenger.css are already used.
Ah, I missed that.
Assignee | ||
Comment 4•1 year ago
|
||
Addressed review comments.
Attachment #9105764 -
Attachment is obsolete: true
Attachment #9105799 -
Flags: review+
Assignee | ||
Updated•1 year ago
|
Keywords: checkin-needed-tb
Updated•1 year ago
|
Target Milestone: --- → Thunderbird 72.0
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/8aa9c7cbd3f9
Port bug 1516091: remove sidebarheader element, convert to <box class="sidebar-header">. r=Paenglab
You need to log in
before you can comment on or make changes to this bug.
Description
•