The attached patch moves the option for custom dates to the first place of the select lists, which makes the availability of this feature more obvious.

Comments

xano’s picture

StatusFileSize
new1.03 KB

Attached a slightly more readable patch.

Status: Needs review » Needs work

The last submitted patch failed testing.

lilou’s picture

Status: Needs work » Needs review
catch’s picture

Component: system.module » usability

Moving to UX queue, and can we have a screenshot?

xano’s picture

The only difference is that the 'Custom format' option has been moved to the top of the select list to prevent it from being overlooked.

yoroy’s picture

AFAIK there is no 'select list' in node/add apart from the one in menu item settings, so it's still not clear what you are saying. So, can we have a screenshot please?

Also, can you tell us *why* you think this particular form should be more prominent?

xano’s picture

StatusFileSize
new60.07 KB
new59.33 KB

I never said I was talking about the node/add form. This is about Administer > Site configuration > Date and time. And why? To prevent the option form being overlooked. I know from several people that they simple didn't notice this option until somebody else told them it was there. The predefined formats kind of 'block' it from view.

catch’s picture

fwiw, I was looking for this the other day, and it took me a while to find, so I think it makes sense putting it at the top. While it's an 'advanced' option, you can't do any harm in there either, and it's easy to back out and use one of the pre-defined formats.

yoroy’s picture

See? screenshots help! :) Seems like a helpfull change to me.

dries’s picture

Putting it at the top helps but it is still not 100% -- if you listen to your gut feeling, you'll feel it is still somewhat awkward. I'm happy to commit this for now, but maybe we can think of an even better fix?

xano’s picture

What about dumping the select list and using a custom date field that accepts tokens that are automatically converted to PHP's date() syntax if the form is being saved? I think this would make everything simpler for most users. Those tokens are much more readable than date()'s syntax, but they do take up a lot more space: d versus [day2digits], for instance. It's just an idea that popped in my mind just now.

catch’s picture

Component: usability » base system
Status: Needs review » Needs work
Issue tags: +Usability

Much bigger issues being dealt with, but IMO stella's patch at http://drupal.org/node/318008#comment-1192626 handles this issue quite well (the comment I linked to has screencasts) - which persuades me the custom option should move outside the select list altogether.

xano’s picture

IMO Stella's patch adds extra clutter. Dries is right, but after two months I still cannot come up with a better idea than my patch in #1.

drupal_was_my_past’s picture

Status: Needs work » Closed (duplicate)
xano’s picture

Assigned: xano » Unassigned