Comment 1 Aaron Sowry 2013-09-11 13:09:19 CEST

This is because the re-order buttons are <input> tags of type "img", which according to the HTML spec[1] do not send a "value" in their query; they send the coordinates of the click on the image. Our server-side code looks for a "value" key, and it's not finding one.

We could possibly reimplement the buttons as actual <button>s, containing an <img> instead.

[1] http://www.w3.org/TR/html4/interact/forms.html#h-17.4.1

Comment 2 Aaron Sowry 2013-09-11 13:35:14 CEST

r27867.

Comment 3 Henrik Andersson 2013-10-11 09:23:44 CEST

Tested using Chrome 30, Firefox 24 and IE11. Works as expected.

Comment 4 Pierre Ossman 2013-10-16 14:48:13 CEST

They may work now, but they've been changed to much bigger buttons instead of the smaller icons used before.

Comment 5 Pierre Ossman 2013-10-18 15:56:52 CEST

Going back to an image control is difficult as we have to encode everything in the "name" attribute. Possible, but messy. So the button approach is probably the best solution, but we need to make sure it only renders the image. r28061 uses CSS to kill off the decorations provided by the browser.

Comment 6 Aaron Sowry 2013-10-21 13:21:58 CEST

Had a quick look and am happy with the solution, resolving bug for Henrik to test.

Comment 7 Henrik Andersson 2013-10-21 14:07:21 CEST

Im also happy with the change of appearance, and those buttons work as expected.