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

Goba drupal-devel at drupal.org
Fri Jan 21 14:11:09 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

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.

Goba



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).


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





More information about the drupal-devel mailing list