Closed
Bug 668958
Opened 13 years ago
Closed 13 years ago
#calendar-status-todaypane-button should use the aero icon
Categories
(Calendar :: Lightning Only, defect)
Calendar
Lightning Only
Tracking
(Not tracked)
RESOLVED
FIXED
1.0b5
People
(Reporter: Paenglab, Assigned: Paenglab)
Details
Attachments
(1 file, 2 obsolete files)
489 bytes,
patch
|
bv1578
:
review+
|
Details | Diff | Splinter Review |
As the summary says the #calendar-status-todaypane-button should use the aero icon
Assignee | ||
Comment 1•13 years ago
|
||
Choose with lightning-aero.css the aero icon when under Vista/Win7
The date number on the icon appears misplaced 1 pixel to the left with the aero theme. It's not a problem of the new icon, it happens with the old one too. This code in the same lightning-aero.css file of this patch fixes the position: #calendar-status-todaypane-button > stack > .toolbarbutton-day-text { margin-left: 1px; } it could be added to the patch for this bug, or does the issue need a specific bug?
Assignee | ||
Comment 3•13 years ago
|
||
Patch with Decathlon's proposal.
Attachment #543582 -
Attachment is obsolete: true
Attachment #543989 -
Flags: review?(philipp)
Attachment #543582 -
Flags: review?(philipp)
Updated•13 years ago
|
Attachment #543989 -
Attachment is patch: true
Attachment #543989 -
Attachment mime type: text/x-patch → text/plain
Attachment #543989 -
Flags: review?(philipp) → review?(bv1578)
Comment on attachment 543989 [details] [diff] [review] Give the aero icon and position the date number r+ ;-) There are two icons that don't use the aero theme yet i.e. those in the "write" drop-down button in the main toolbar.
Attachment #543989 -
Flags: review?(bv1578) → review+
Assignee | ||
Comment 5•13 years ago
|
||
(In reply to comment #4) > There are two icons that don't use the aero theme yet i.e. those in the > "write" drop-down button in the main toolbar. I'll open a new bug.
Keywords: checkin-needed
Assignee | ||
Updated•13 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 6•13 years ago
|
||
Because this patch isn't checked-in and the change is minimal I added the icons in the "write" drop-down button.
Attachment #543989 -
Attachment is obsolete: true
Attachment #545064 -
Flags: review?(bv1578)
Comment on attachment 545064 [details] [diff] [review] Give the aero icons It works fine. r+
Attachment #545064 -
Flags: review?(bv1578) → review+
Keywords: checkin-needed
Comment 8•13 years ago
|
||
http://hg.mozilla.org/comm-central/rev/408ad8acf001
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Trunk
Comment 9•13 years ago
|
||
Thanks for taking care of the checkin! Could you also check in on comm-miramar, both on the default and COMM50_20110620_RELBRANCH branches (and set the target milestone to 1.0b5 then)? If you don't have time I'll take care within the next few days
Keywords: checkin-needed
Comment 10•13 years ago
|
||
Backported to comm-miramar <http://hg.mozilla.org/releases/comm-miramar/rev/3afe9bcae179> and <http://hg.mozilla.org/releases/comm-miramar/rev/45face4cd4d6>
You need to log in
before you can comment on or make changes to this bug.
Description
•