I tried enabling the Week Numbers option on my Mini Calendar display, but doing so causes a strange rendering bug. It adds a new unlabeled column on the left of the table, but it doesn't populate that column with the week number links. The entire column goes totally unrendered, in fact, which causes all the day links to be shifted one column to the left.

I'm going to look into a way to fix this, since I need it to work for my site's calendar.

CommentFileSizeAuthor
#1 calendar_mini-calendar-week-numbers-fix-2008554-1.patch1.3 KBcoredumperror
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

coredumperror’s picture

coredumperror CreditAttribution: coredumperror commented
Status: Active » Needs review
FileSize
calendar-mini-calendar-week-numbers-fix-2008554-1.patch1.3 KB

Here's a patch that adds the week numbers to the mini-calendar month view. I'm not certain that it's 100% the right way to do this, but it seems to work fine.

odegard’s picture

odegard CreditAttribution: odegard commented
Status: Needs review » Reviewed & tested by the community

I can confirm the symptom, the patch solves the problem, and the code looks solid. Thanks!

toaijala’s picture

toaijala CreditAttribution: toaijala commented

Can confirm it works perfectly for me aswell.

fizk’s picture

fizk CreditAttribution: fizk commented
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Committed, thanks!

  • Commit aa33327 on 7.x-3.x authored by coredumperror, committed by fizk: 
    #2008554: Properly display week numbers in mini calendar view.
coredumperror’s picture

coredumperror CreditAttribution: coredumperror commented

Oh that's very good to hear. Thank you!

Status: Fixed » Closed (fixed)

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