Side navigation
#2229 closed bug (fixed)
Opened January 24, 2008 05:10AM UTC
Closed March 05, 2008 04:26AM UTC
confusion between datepicker_currentDay class and datepicker_today class
Reported by: | Zola | Owned by: | kbwood |
---|---|---|---|
Priority: | minor | Milestone: | 1.2.3 |
Component: | ui | Version: | 1.2.2 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
minor-but-annoying.
in the ui.css file, there seems to be a class intended to let you use a different background to differentiate between "today" and "day selected"
I set the current_day to light purple and the datepicker_today class to blue.
See the attached image:
When you open up the calendar, you see that today, January 23rd, is in purple like it's supposed to be. (left calendar view).
However, when I select a day, THAT day is then purple and the current day takes on the blue color (right calendar view)
If I swapped the two around, it still has the same problem, just in reverse--the current day would be blue, and change to purple while the selected day would turn to blue.
I think it's just class confusion--those two class names are very similar and it's hard to tell which class is for today and which is for the selected day, and clearly, that's what got messed up in the code.
Maybe it would be better to rename datepicker_today class to "datepicker_SelectedDate"? It would be nice to be able to use two different colors and have the current day remain the same color and have the selected day a different color.
Attachments (2)
Change History (2)
Changed January 29, 2008 12:30AM UTC by comment:1
owner: | paul → kbwood |
---|---|
status: | new → assigned |
Changed March 05, 2008 04:26AM UTC by comment:2
resolution: | → fixed |
---|---|
status: | assigned → closed |
The latest version applies both datepicker_today and datepicker_currentDay if applicable. Previously currentDay overrode today. Also, currentDay is only applied if a date is actually selected, not when the input is blank and the default date is highlighted.
There are three CSS classes involved (see the attached image):