Bugzilla – Bug 4719
Profile reordering does not work
Last modified: 2013-10-24 10:59:17
You need to
before you can comment on or make changes to this bug.
If you press any of the "Reorder" buttons in the ThinLinc Web Administration,
This is because the re-order buttons are <input> tags of type "img", which
according to the HTML spec 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
Tested using Chrome 30, Firefox 24 and IE11. Works as expected.
They may work now, but they've been changed to much bigger buttons instead of
the smaller icons used before.
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.
Had a quick look and am happy with the solution, resolving bug for Henrik to
Im also happy with the change of appearance, and those buttons work as