Closed GoogleCodeExporter closed 8 years ago
hope this helps a bit with my thinking...
on #2 -- looks to be a configurable item if an option (daysToDisplay) versus
numeral 7
-- also the use of index 'i' to feed from the daysOfWeek string array versus getDay
#247 (use of numeral 7)
for(var i=1 ; i<=7; i++) {
calendarHeaderHtml += "<td class=\"day-column-header day-" + i + "\"></td>";
}
#258 (use of numeral 7)
<td colspan=\"7\">\
#282 (use of numeral 7)
for(var i=1 ; i<=7; i++) {
calendarBodyHtml += "<td class=\"day-column day-" + i + "\"><div
class=\"day-column-inner\"></div></td>"
}
#420 (dayNames is using index 'i' versus getDay() value)
$(this).html(dayNames[i] + "<br/>" + monthNames[currentDay.getMonth()] + ", " +
currentDay.getDate() + ", " + currentDay.getFullYear());
Original comment by robert.e...@gmail.com
on 3 Jun 2009 at 4:54
also just saw : prevWeek and nextWeek could be prevPeriod and nextPeriod based
on
MILLIS_IN_WEEK becoming MILLIS_IN_PERIOD
MILLIS_IN_PERIOD = MILLIS_IN_DAY * DaysInPeriod
Thanks...
Original comment by robert.e...@gmail.com
on 3 Jun 2009 at 4:56
hacked in the required changes for it to work for me... a few issues remain...
too
many days in a period for a given horizontial width does not look good <g>
but works: and thanks again for this great control, you rock brother...
cheers,
robert
Original comment by robert.e...@gmail.com
on 3 Jun 2009 at 5:30
Attachments:
Great stuff Robert. I've emailed you to check if you have the latest changes
as i'd
like to incorporate this into the next release.
Original comment by robmo...@gmail.com
on 8 Jun 2009 at 12:08
Original comment by robmo...@gmail.com
on 8 Jun 2009 at 12:09
After some more thought i've decided to limit the initial scope of this
feature. I'm
going to keep the calendar to a maximum of 7 days but allow any number of days
within
the week to be shown or hidden so as to facilitate a working week for instance.
Horizontal display of periods longer than 7 days introduce layout complexities
and
IMO introduce usability issues unless the user has very high screen
resolutions. I'd
be inclined to follow the lead of google calendar for implementations of longer
than
a week. If people require the display of durations longer than a week I
recommend
using the full calendar plugin which gives an excellent monthly calendar view -
http://arshaw.com/fullcalendar/
Original comment by robmo...@gmail.com
on 17 Jun 2009 at 11:38
This enhancement addresses the disable user interaction with the calendar
through a
calendar wide 'readonly' config option. It will be in the next 1.2 release.
Original comment by robmo...@gmail.com
on 19 Jun 2009 at 3:58
Original issue reported on code.google.com by
robert.e...@gmail.com
on 3 Jun 2009 at 1:35