Closed Bug 894501 Opened 8 years ago Closed 8 years ago

Move close-sidebar.png into calendar

Categories

(Calendar :: Calendar Frontend, defect)

x86_64
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: dao, Assigned: Paenglab)

References

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #894490 +++
Attached patch patchSplinter Review
This patch changess the use of close-sidebar.png to close.png which is widely used like in tabs or also in unifinder-closer. The cross is a little bit lighter than the one from close-sidebar.png but as close.png is used on other places it makes also sense to use the same colors on all together visible places.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #776563 - Flags: ui-review?(philipp)
Attachment #776563 - Flags: review?(philipp)
Comment on attachment 776563 [details] [diff] [review]
patch

I already asked this in bug 894490, but will close.png stay when australis lands?
Attachment #776563 - Flags: ui-review?(philipp)
Attachment #776563 - Flags: ui-review+
Attachment #776563 - Flags: review?(philipp)
Attachment #776563 - Flags: review+
(In reply to Philipp Kewisch [:Fallen] from comment #2)
> Comment on attachment 776563 [details] [diff] [review]
> patch
> 
> I already asked this in bug 894490, but will close.png stay when australis
> lands?

Yes.
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/835b680fda7e
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 2.7
You need to log in before you can comment on or make changes to this bug.