User:Salvatore Ingala/Review 2


 * I'd like to be able at least to see the numeric value of track bars. Ideally, it should be possible to enter from keyboard.
 * Added something similar to tooltips in ; keyboard input needs this jQuery.UI bug to be fixed to be useful. Salvatore Ingala 17:41, 17 August 2011 (UTC)
 * "Settings saved successfully" message box is too intrusive, better use a non-modal confirmation.
 * Save button should be clickable only if settings are actually changed.
 * Use standard buttons in dialog, otherwise Mac users might want to apply tar and feathers to you.
 * Re-check proxy caching headers for Gadget API modules and module visibility for RL modules to avoid leaking private data.
 * After your recent commits I don't see the link to gadget prefs anymore, even after updating HotCat.preferences.

Errors

 * Should be fixed in . Salvatore Ingala 09:08, 17 August 2011 (UTC)

If we try to save prefs with invalid JSON:


 * Should be fixed as per this change in . Salvatore Ingala 09:08, 17 August 2011 (UTC)