Hi Raluca,
Looks good. Some remarks:
*__A__ *
div.datepicker table tbody tr td.nclick_outbound,
div.datepicker table tbody tr td.nclick_outbound:hover {
color: $theme.backgroundSecondaryColor;
}
should be
color: $theme.textSecondaryColor;
because it's a text color.
*__B__*
every "0px" should be "0" simple;
*__C__*
div.datepicker {
font-family: arial;
font-size: 10px;
}
our default is
font-family:sans-serif;
and also, we should use relative font sizes (em, %) instead of "px".
*__D__*
and if you want me to be very picky :P you could use
font-weight: 400; instead of normal
*__E__*
Also the inline styles, like for:
<datepicker-mydate_id-prev style="cursor: pointer;"> should be integrated in
the css.
*__F__*
you should remove the
filter:alpha(Opacity=0); for the IE
Hope this review is helpful,
Caty
On Fri, Apr 16, 2010 at 12:53, Raluca Stavro <[email protected]>wrote:
> Hello devs,
>
> I would like to contribute to the DatePicker application.
> Currently, the DatePicker stylesheet does not take in account the
> color theme. This is why I created a Jira issue (XCONTRIB-109) and I
> attached a patch to it. You will see some screenshots with the new
> styles applied on the DatePicker table.
> Can someone please give me the right to commit those styles?
>
> Raluca.
> _______________________________________________
> devs mailing list
> [email protected]
> http://lists.xwiki.org/mailman/listinfo/devs
>
_______________________________________________
devs mailing list
[email protected]
http://lists.xwiki.org/mailman/listinfo/devs
