This really applies to both the 6.2 and 5.2 versions, but now that the new jCalendar popup has been added to the code, I see a few things that could be improved.

One is that we should probably display the teaser rather than the full node on the popup, or offer a settings option to choose one or the other. If a node has a lot of information, it isn't going to look good in the popup.

CommentFileSizeAuthor
#4 jcalendar.patch641 byteswill_in_wi
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

KarenS’s picture

KarenS CreditAttribution: KarenS commented
Component: Code » jCalendar popup

I added a new component where we can mark issues for this module.

will_in_wi’s picture

will_in_wi CreditAttribution: will_in_wi commented

This is probably a good idea. Currently, if a body is too long, it will scroll. Is this enough?

KarenS’s picture

KarenS CreditAttribution: KarenS commented

I suggest we either default to the teaser or find a way to provide an option of which to display (and it might take some thought to figure out the best place to offer that option). I think many or most nodes will be far too big for that little box, and will get badly displayed, even with a scroll bar.

will_in_wi’s picture

will_in_wi CreditAttribution: will_in_wi commented
FileSize
jcalendar.patch641 bytes

This is the fix to switch it to teaser view. The api for this has not changed between d5 and d6 so the fix will work on both.

KarenS’s picture

KarenS CreditAttribution: KarenS commented
Status: Active » Reviewed & tested by the community

Looks fine, go ahead and commit it. We can come back later and find away to provide more options, but this seems like a sane default value.

will_in_wi’s picture

will_in_wi CreditAttribution: will_in_wi commented
Status: Reviewed & tested by the community » Fixed

Fixed in both d5 and 6.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.

gooddesignusa’s picture

gooddesignusa CreditAttribution: gooddesignusa commented

Could we get this to use a build mode from Display Suite?

ipublic’s picture

ipublic CreditAttribution: ipublic commented

subscribe

it would be nice, if we could change/set the build mode for the popup because we have to change the content for the popup but need functional teasers as well, which should be different to the calender popup