The deadline field would be a lot easier to use if there was a date picker widget. These date pickers are popping up all over the web. This is not exactly a AJAX issue but it would be easier to solve with whatever js framework is included for the AJAXification. For an example date picker widget take a look at: http://www.frequency-decoder.com/demo/fd-date-picker/
This would be very easy to do now that we have a date-picker widget for date/time fields. pyrzak's students' research indicated that the lack of this date-picker was a problem. (Largely because it's inconsistent with any date/time custom field that appears on enter_bug.cgi.)
Priority: -- → P1
Created attachment 410459 [details] hack to display calendar button next to the deadline This is a hack for 3.4.2. You can use it in custom templates to add the calendar button next to "deadline". Thanks for a great piece of software!
Created attachment 411765 [details] [diff] [review] Tip I love it when I go to submit a patch and someone has already done so. :D This patch is against tip. I have tested this on tip as well.
Frank, next time be sure to request review and upload a diff file (patch). It makes it a lot easier for the devs to apply and test. ;)
Created attachment 411776 [details] [diff] [review] Tip - V2 We should have it on bug entry and update-multiple as well.
Duplicating code is not good. We should put it in a shared template (there is one already, probably), and PROCESS it.
Created attachment 411784 [details] [diff] [review] Tip - V3 Moved processing to a block in bug/time.html.tmpl
Sorry for using the incorrect procedure. I'll try to remember this next time... Cheers
Comment on attachment 411784 [details] [diff] [review] Tip - V3 Why don't you just use bug/field.html.tmpl and fake the type or something?
Yeah, in fact, just set the "type" of the deadline field in the database and then just use bug/field.html.tmpl along with bug_fields.deadline.
But deadline is a Date type, not a datetime. To my knowledge it is the only one of this type. Why go through all the trouble of defining a new type?
(In reply to comment #12) > But deadline is a Date type, not a datetime. To my knowledge it is the only one > of this type. Why go through all the trouble of defining a new type? It doesn't need a new type. The DATETIME type will work just fine for it.
Attachment #411784 - Flags: review?(LpSolit) → review-
Created attachment 425727 [details] [diff] [review] v4 Here we go! Pretty straightforward.
Comment on attachment 425727 [details] [diff] [review] v4 OK, I see what you meant now. This looks good.
Attachment #425727 - Flags: review?(LpSolit) → review+
Thanks for the review, ghendricks! :-) Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/ modified Bugzilla/Bug.pm modified Bugzilla/Field.pm modified skins/standard/show_bug.css modified template/en/default/bug/edit.html.tmpl modified template/en/default/bug/create/create.html.tmpl modified template/en/default/list/edit-multiple.html.tmpl Committed revision 6999.
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Added to the release notes in bug 604256.
You need to log in before you can comment on or make changes to this bug.