293390 - layout issues on date/time picker to use system colors and clean up layout

Status: RESOLVED FIXED

(Calendar :: General, defect)

Description

[Andrew Douglas]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b2) Gecko/20050424 Firefox/1.0+
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b2) Gecko/20050424 Firefox/1.0+

The date/time pickers were using non-system defined colors in their css, which
needs to be fixed before landing in toolkit. Some text on the time picker wasn't
being centered.

Reproducible: Always

Comment 1

[Andrew Douglas]

Attachment: fix css and some centering issues in the timepicker's xbl

should now use system defined colors for everything (well, if you say that
-moz-Dialog is a system defined color). It should now fit in well with toolkit.
Note: only classic skin was impacted as this is what will be used to move to
toolkit.

Also: some files from the datepicker cleanup can be removed out of
/skin/XXX/datetimepickers/. Namely datepicker.css and timepicker.css.

Comment 2

[Andrew Douglas]

ack. this also pulled my "clean up" of an if statement inside initTimeFormat. I
can break that out into a separate patch if necessary. Sorry, trying to do too
many things at once.
-Andrew

Comment 3

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Comment on attachment 182970 [details] [diff] [review]
fix css and some centering issues in the timepicker's xbl

Looks good to me..

r=vladimir

Comment 4

[gekacheka]

I think the following "budy" specialization can be removed ("budy" appears
nowhere else according to lxr, and an otherwise identical "busy" specialization
appears a few lines later).

.minimonth-day[budy="true"] {
  font-weight : bold;
}


You may also be interested in minimonth patches in bug 286841 and bug 264150.

Setting status: confirmed (was unconfirmed)

Comment 5

[Andrew Douglas]

Comment on attachment 182970 [details] [diff] [review]
fix css and some centering issues in the timepicker's xbl

mvl checked in 5/10/2005. Leaving bug open for some additional cleanup work.

Comment 6

[Joey Minta]

(In reply to comment #5)
> (From update of attachment 182970 [details] [diff] [review] [edit])
> mvl checked in 5/10/2005. Leaving bug open for some additional cleanup work.
> 
Andrew, can you clarify what else needs to be done for this bug to be closed?

Comment 7

[Andrew Douglas]

Sorry, I should have been more specific as I don't recollect what else was being planned at that point. I don't have any pending layout issues here so unless someone else knows of something, it can be closed out.
-Andrew

Comment 8

[Joey Minta]

(In reply to comment #7)
> Sorry, I should have been more specific as I don't recollect what else was
> being planned at that point. I don't have any pending layout issues here so
> unless someone else knows of something, it can be closed out.
> -Andrew
> 
Thanks for all your hard work.  I'll go ahead and mark this fixed.