Closed
Bug 1600435
Opened 4 years ago
Closed 4 years ago
Calendar "edit" icon looks inconsistent with the rest of Thunderbird
Categories
(Calendar :: General, defect)
Calendar
General
Tracking
(Not tracked)
RESOLVED
FIXED
68.3.1
People
(Reporter: ntim, Assigned: Paenglab)
Details
Attachments
(4 files)
|
23.43 KB,
image/png
|
Details | |
|
16.99 KB,
image/png
|
Details | |
|
5.16 KB,
patch
|
pmorris
:
review+
pmorris
:
approval-calendar-beta+
|
Details | Diff | Splinter Review |
|
5.71 KB,
patch
|
pmorris
:
approval-calendar-esr+
|
Details | Diff | Splinter Review |
Calendar screenshot
It should be using the same icon as the compose icon.
|
Reporter | |
Comment 1•4 years ago
|
||
|
Assignee | |
Comment 2•4 years ago
|
||
The same icon as the TB newmsg icon. Only a bit optimised through SVGOMG to not be a duplicate.
|
|
Assignee | |
Updated•4 years ago
|
Component: Theme → General
Product: Thunderbird → Calendar
|
|
Assignee | |
Comment 3•4 years ago
|
||
Patch for ESR.
Attachment #9112743 -
Flags: approval-calendar-esr?(paul)
|
|
Assignee | |
Updated•4 years ago
|
Attachment #9112742 -
Flags: approval-calendar-beta?(paul)
|
||
Comment 4•4 years ago
|
||
Comment on attachment 9112742 [details] [diff] [review] 1600435-calendar-edit-icon.patch Review of attachment 9112742 [details] [diff] [review]: ----------------------------------------------------------------- Changes look fine, and the new icon looks great when I checked it.
Attachment #9112742 -
Flags: review?(paul)
Attachment #9112742 -
Flags: review+
Attachment #9112742 -
Flags: approval-calendar-beta?(paul)
Attachment #9112742 -
Flags: approval-calendar-beta+
|
|
||
Updated•4 years ago
|
Attachment #9112743 -
Flags: approval-calendar-esr?(paul) → approval-calendar-esr+
|
|
Assignee | |
Updated•4 years ago
|
Keywords: checkin-needed-tb
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/9bb60ca53c7b
Use the newmsg icon as the edit icon in calendar. r=pmorris
|
||
Updated•4 years ago
|
Flags: needinfo?(philipp)
Target Milestone: 72 → 73
|
||
Comment 7•4 years ago
|
||
Target Milestone: 73 → 72
|
|
||
Comment 8•4 years ago
|
||
|
|
||
Updated•4 years ago
|
Target Milestone: 72 → 68.3.1
You need to log in
before you can comment on or make changes to this bug.
Description
•