Closed Bug 558746 Opened 10 years ago Closed 10 years ago

Update Modern theme for videocontrols including HTML5 CSS Transitions

Categories

(SeaMonkey :: Themes, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: neil, Assigned: philip.chee)

References

Details

(Keywords: modern, Whiteboard: [good first bug])

Attachments

(1 file)

For some reason the CSS rules added to toolkit (p/winstripe) videocontrols.css were all jammed together without any blank lines.
Morphing this bug slightly to cover the following:

Bug 489902 - video controls should prefer a specific font name
Bug 520614 - Fix usage of vbox in videocontrols.xml
Bug 521890 - Use CSS Transitions for HTML5 videocontrols
Assignee: nobody → philip.chee
Status: NEW → ASSIGNED
Depends on: 520614, 489902
Keywords: helpwanted
Summary: Update Modern theme for HTML5 videocontrols CSS Transitions → Update Modern theme for videocontrols including HTML5 CSS Transitions
>-  font: 11px sans-serif;
>+  font-size: 11px;
>+  font-family: Helvetica, Arial, sans-serif;

winstripe uses Arial, san-serif;
pinstripe uses Helvetica, san-serif;

OS X ships with both Helvetica and Arial so Helvetica should be selected first.

Windows 7 ships with Arial but not Helvetica out of the box so Arial should be selected first. If the user chooses to install Helvetica then we assume he likes it enough to use it. In either case, they should be better choices than the generic san-serif.
Attachment #448973 - Flags: feedback?(neil)
(In reply to comment #1)
> Bug 489902 - video controls should prefer a specific font name
This landed on 1.9.1.x; should it land on 2.0.x?
Comment on attachment 448973 [details] [diff] [review]
Patch v1.0 sync with toolkit

>+}
>+.statusOverlay[fadeout] {
Nit: Blank line in between.
Attachment #448973 - Flags: review+
Attachment #448973 - Flags: feedback?(neil)
Attachment #448973 - Flags: feedback+
>>+}
>>+.statusOverlay[fadeout] {
> Nit: Blank line in between.
Fixed locally.

Pushed to comm-central
http://hg.mozilla.org/comm-central/rev/59716a0a03cc
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.