Closed
Bug 1697545
Opened 4 years ago
Closed 4 years ago
Let the attachmentToggle in message view animate too
Categories
(Thunderbird :: Theme, task)
Thunderbird
Theme
Tracking
(thunderbird_esr78 wontfix, thunderbird87 wontfix)
RESOLVED
FIXED
88 Branch
People
(Reporter: Paenglab, Assigned: Paenglab)
Details
Attachments
(1 file)
|
1.58 KB,
patch
|
aleca
:
review+
|
Details | Diff | Splinter Review |
The attachmentToggle in composer is animated but the one in message pane not.
| Assignee | ||
Comment 1•4 years ago
|
||
Make it animate too.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #9208166 -
Flags: review?(alessandro)
Comment 2•4 years ago
|
||
Comment on attachment 9208166 [details] [diff] [review]
1697545-attachmentToggle-animate.patch
Review of attachment 9208166 [details] [diff] [review]:
-----------------------------------------------------------------
Nice, thanks for doing this.
A couple of follow ups to this could be:
- Use the HTML <img> tag and remove the XUL <image>
- Use the same CSS rotation for all the other twisty in order to remove the collapse and rtl SVG variations.
If you think those changes could be good :D
Attachment #9208166 -
Flags: review?(alessandro) → review+
| Assignee | ||
Updated•4 years ago
|
status-thunderbird87:
--- → wontfix
status-thunderbird_esr78:
--- → wontfix
Keywords: checkin-needed-tb
Target Milestone: --- → 88 Branch
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/84812fd86386
Let the attachmentToggle in message view animate too. r=aleca DONTBUILD
You need to log in
before you can comment on or make changes to this bug.
Description
•