Ticket #6260 (closed enhancement: fixed)

Opened 4 months ago

Last modified 4 months ago

[PATCH] Show week numbers in calendar

Reported by: terje Assigned to: doki_pen
Priority: normal Component: DateFieldPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11

Description

Javascript from http://dev.jqueryui.com/changeset/3004#file48

Attachments

weeknumbers.diff (90.6 kB) - added by terje on 11/25/09 18:06:20.
weeknumbers2.diff (90.3 kB) - added by terje on 11/25/09 20:20:02.
Updated with correct fix

Change History

11/25/09 18:06:20 changed by terje

  • attachment weeknumbers.diff added.

11/25/09 19:46:38 changed by anonymous

Where did this change come from and what does it fix?

< if(firstDay>4&&(checkDate.getDay()7)<firstDay-3){return 1}}}return Math.floor(((checkDate-firstMon)/86400000)/7)+1}, ---

if(firstDay>4&&(checkDate.getDay()7)<firstDay-3){return 1}}}return Math.floor(Math.round((checkDate-firstMon)/86400000)/7)+1},

11/25/09 19:50:30 changed by doki_pen

Sorry, I meant this: 

 <
 if(firstDay>4&&(checkDate.getDay()||7)<firstDay-3){return 1}}}return
 Math.floor(((checkDate-firstMon)/86400000)/7)+1},
 ---
 >
 if(firstDay>4&&(checkDate.getDay()||7)<firstDay-3){return 1}}}return
 Math.floor(Math.round((checkDate-firstMon)/86400000)/7)+1},

(follow-up: ↓ 4 ) 11/25/09 19:50:36 changed by doki_pen

  • status changed from new to assigned.

11/25/09 20:20:02 changed by terje

  • attachment weeknumbers2.diff added.

Updated with correct fix

(in reply to: ↑ 3 ) 11/25/09 20:23:06 changed by terje

Replying to doki_pen: Sorry that was a quick hack to fix an off-by-one error New attachment includes correct fix from http://dev.jqueryui.com/changeset/2423

11/25/09 20:52:47 changed by doki_pen

  • status changed from assigned to closed.
  • resolution set to fixed.

(In [7183]) Week numbers.

and jquery-ui fixes related to week numbers

fixes #6260 thanks terje

Add/Change #6260 ([PATCH] Show week numbers in calendar)




Change Properties
Action