Skip to main content

This site requires you to update your browser. Your browsing experience maybe affected by not having the most up to date version.

Archive

Our old forums are still available as a read-only archive.

Moderators: martimiz, Sean, biapar, Willr, Ingo, simon_w

Change to CalendarDateField


Reply

6 Posts   1619 Views

Avatar
Sam

23 May 2007 at 11:02am Administrator, 679 Posts

Hey guys,

I'm currently working on a change to CalendarDateField, so that the calendar is accessible by clicking a calendar icon to the right of the field, rather than the field itself. This is because it's currently really hard to type in a value without using the calendar.

My question is - should this be implemented as an optional part of CalendarDateField, turned off by default, or should I change the default behaviour?

Avatar
Sean

23 May 2007 at 8:32pm Forum Moderator, 921 Posts

I say this should be default. I've already thought the calendar popup is a bit unusable, especially as you said Sam - it's not clear that you can fill in the field yourself.

My two cents. :-)

Sean

Avatar
Willr

23 May 2007 at 9:47pm Forum Moderator, 5511 Posts

Im leaning towards having it default :D. Funny I always thought there was meant to be a icon to the right as the calendar textbox seems shorter then the others..

Avatar
Sigurd

24 May 2007 at 12:45pm Forum Moderator, 628 Posts

I vote "default"

Additionally, almost all of the time I use the date field, I am choosing today, or something very close, so it would make sense for the value inside the textbox to show today. I think having this as default too, but with the ability to disable the day's default.

As it stands, I sometimes find myself writing 2006 and wondering why it doesn't show up :P

Avatar
Sam

27 May 2007 at 12:00pm Administrator, 679 Posts

I've made it the default, and put a note for Andrew to include backward-compatability issues in the 2.0.2 changelog.

Sig - defaulting to today can get tricky when you want to clear the value of the field out. In short - it's one of those things you wind up swearing at in lots of situations. The calendar has today highlighted, so setting to today is a 2-click process.

Avatar
Matt

30 May 2007 at 11:56pm Core Development Team, 84 Posts

Would be cool to get some simple strtotime working in here - we had quite an advanced implementation in SilverStripe 1 that is probably not entirely necessary here, but changing words like 'today', 'tomorrow', 'yesterday', the days of the week etc would be cool.