#11436 closed enhancement (fixed)
Add datepickers to relative time fields on the Query page
Reported by: | Ryan J Ollos | Owned by: | Ryan J Ollos |
---|---|---|---|
Priority: | normal | Milestone: | 1.1.2 |
Component: | query system | Version: | |
Severity: | normal | Keywords: | jqueryui datepicker |
Cc: | Branch: | ||
Release Notes: |
Added datepickers to time fields with the Relative format on the Query page. This change restores datepickers to the Created and Modified fields. |
||
API Changes: | |||
Internal Changes: |
Description
Adding datepickers for relative time fields on the query page will restore them to the created and modified fields, as mentioned in comment:80:ticket:1942 and comment:88:ticket:1942. On 1.0-stable the created and modified fields have datepickers.
Attachments (0)
Change History (3)
comment:1 by , 11 years ago
Component: | general → query system |
---|---|
Milestone: | next-dev-1.1.x → 1.1.2 |
Release Notes: | modified (diff) |
Resolution: | → fixed |
Status: | new → closed |
comment:2 by , 11 years ago
Now that I think about this more, I don't think it makes sense to add a datepicker to batch input fields with format Relative. I think it only make sense to add a datepicker when there is a pair of input fields like the Query filters provide.
-
trac/htdocs/js/query.js
diff --git a/trac/htdocs/js/query.js b/trac/htdocs/js/query.js index cabd83c..dfe4ea1 100644
a b 333 333 focusElement = createText(inputName, 42).addClass("time"); 334 334 if (property.format == "datetime") { 335 335 focusElement.datetimepicker(); 336 } else if (property.format == "date" || 337 property.format == "relative") { 336 } else if (property.format == "date") { 338 337 focusElement.datepicker(); 339 338 } 340 339 td.append(focusElement);
Committed to trunk in [12423].