Tyron Madlener a écrit :
On Mon, May 30, 2011 at 6:47 PM, Marc Delisle marc@infomarc.info wrote:
Tyron Madlener a écrit :
Since I always have a love for details, here's 3 small suggestions that maybe can be merged into the master branch. They are all pushed to my fork (http://repo.or.cz/w/phpmyadmin/tyronm.git):
- Commit a0302c666c1bf9f9715364a6301238b474aedac1
Removes timepicker.js. It is already included with the jquery ui. The file was included only in tbl_change.php and tbl_select.php
Tyron, I don't see a timepicker in jQuery UI's Datepicker. It only handles the date, not the time.
Oh right. But it is not being used. tab_change.js uses PMA_addDatepicker() from functions.js which uses $.datepicker(), not $.timepicker() from what I can see.
Tyron,
js/jquery/timepicker.js is currently used. To prove it, rename the file and try to pick a time when inserting or searching on a table with a time column: it won't work.
And tbl_select.js interestingly doesn't initialize any date/timepicker widget at all.
Shall I try and implement it instead?