CUNY Academic Commons

1. Use two dropdowns instead. The hour dropdown would somehow contain the am/pm distinction.

Attached is a IRL example of this (1_Two Dropdowns.jpg). Looks a little strange, but clear enough, I think.

2. Use a Google Calendar-inspired option: text-entry field that, when clicked, shows a list of half-hour increments to choose from. Values that are not on the half-hour can be entered by hand.

This would be awesome/ideal! But it if's cumbersome to create, not a big enough feature to spend too much time on.

I'd like to add an option 3:
Pre-filled text that is editable with a dropdown for AM/PM (see 3_Editable text w dropdown). The colon is static but the numbers on either side can be edited manually. I think this is more eloquent that option #1 but possibly be harder to create.

I defer to you based on coding-complexity, Boone. Any of the 3 would be cool, with my preference order being 2, 3, 1.

http://www.jqueryscript.net/time-clock/Google-Calendar-Like-jQuery-Data-Time-Range-Picker-datepair.html Looks like we can probably steal this for the Google Calendar option. MIT licensed. Ray, you want to have a look? Again, this is potentially an upstream improvement - my opinion is that it's much better than what's currently in Event Organiser, but I suppose that's subject to opinion.

I've submitted a pull request to the Event Organiser repo with the new Datepair.js / timepicker JS.

Here's what it looks like:

Going to set this ticket to "Hold" for now until Event Organiser decides to merge it.

I would say 'no' but I would implement this if the community needed it.

Having glanced over the PR, I suspected the same thing. I just wanted to check :) I'm going to move this to the 2.0 milestone, and we'll move it back to a minor release once EO 3.0 has been released.

For now, I've ensured that our customized version of timepicker is running from BPEO instead of EO (see BPEO commit 498e989.

However, this means we are also forking EO's event.js, so we'll have to keep track of any upstream changes.

This is available for testing on cdev.