[drupal-devel] [feature] Enable themeing of blank calendar days

Goba drupal-devel at drupal.org
Sun Jan 23 00:10:39 UTC 2005

 Project:      Drupal
 Version:      cvs
 Component:    archive.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Goba
 Reported by:  Goba
 Updated by:   Goba
 Status:       patch
 Attachment:   http://drupal.org/files/issues/Drupal-allow-individual-blank-day-styles.patch (2.88 KB)

As Steven suggested, here is a version, which always prints out
individual table cells for blank days, so it is possible to style them
differently in CSS, but retains the well-known calendar format with
dropping the borders on the blank days, and falling back on
border-collapsing (this is why the table border needs to be

Also included in this patch are some simple CSS optimizations to use 0
instead of 0em and 0px.


Previous comments:

January 15, 2005 - 20:30 : Goba

Attachment: http://drupal.org/files/issues/drupalcal.png (16.16 KB)

I would like to have a calendar which has stlyes for every blank day, as
opposed to what is supported by current Drupal HEAD. It should look
something like:

I see that it would only be possible if we make the blank days
themeable. It would be possible to make the whole calendar themeable,
but that would open up a lot more stuff, since then the calendar code
would need to be abstracted out from the archive table assembler code.
BTW this is a Hungarian Drupal setup, with monday being the first day,
and the calendar is for 2005 January.


January 15, 2005 - 20:38 : Goba

Attachment: http://drupal.org/files/issues/Drupal.allow.themeing.blank.days.patch (1.28 KB)

Here is a proposed patch, which simply makes the blank days themeable,
so that I can use str_repeat(" \n", $blankdays) in my theme.


January 21, 2005 - 14:58 : Goba

What is this patch is waiting for? :)


January 21, 2005 - 15:07 : Steven

I'm leaning towards making the whole calendar themable rather than just
the blank days. It seems a bit too shallow. Another thing to consider
is to simply output the blank days as individual cells, and use CSS to
hide their border in the default themes (it uses border-collapse:
collapse; so I think it wouldn't require special handling of the
first/last blank cell).


January 21, 2005 - 15:11 : Goba

Good to get input on this :) Either way is fine with me, but making the
whole calendar themeable is a lot more work in the code, since now
logic and display are very much intermixed in the calendar code. It
would be nice, if the CSS solution would work.

View: http://drupal.org/node/15655
Edit: http://drupal.org/project/comments/add/15655

More information about the drupal-devel mailing list