since javascript is obligatory anyway there is no need to confirm a sort by key request. On the contrary many other similar select boxes make use of "onChange" submit, e.g., language select... (patch for pma 2.11 attached)
I'm not so sure that javascript is obligatory in phpMyAdmin. I seem to remember that we take (took?) care in ensuring that it works without javascript. If you check libraries/display_select_lang.lib.php, you'il see a <noscript> section that displays a Go button.
Of course we can add some onchange statements, provided there is an submit button alternative.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
>Of course we can add some onchange statements, provided there is an submit button alternative.
ok, thats an option, but i thouhgt to remember some statements that using pma without js enabled was useles.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Logged In: YES
user_id=1383652
Originator: YES
patch for pma 2.10 added
File Added: SortByKeyOnChange 2.10.patch
Logged In: YES
user_id=210714
Originator: NO
I'm not so sure that javascript is obligatory in phpMyAdmin. I seem to remember that we take (took?) care in ensuring that it works without javascript. If you check libraries/display_select_lang.lib.php, you'il see a <noscript> section that displays a Go button.
Of course we can add some onchange statements, provided there is an submit button alternative.
Logged In: YES
user_id=1383652
Originator: YES
>Of course we can add some onchange statements, provided there is an submit button alternative.
ok, thats an option, but i thouhgt to remember some statements that using pma without js enabled was useles.
Logged In: YES
user_id=1383652
Originator: YES
File Added: SortByKeyOnChange 2.10withNoScript.patch
patch with <noscript> option</noscript>
new 2.11 patch with< NoScript> option
Logged In: YES
user_id=1383652
Originator: YES
<noscript> modified patches attached
File Added: SortByKeyOnChange 2.11withNoScript.patch
Logged In: YES
user_id=210714
Originator: NO
Merged in subversion, thanks.