[Bug 555221] New: admin module: excessive "save" buttons
http://bugzilla.novell.com/show_bug.cgi?id=555221 Summary: admin module: excessive "save" buttons Classification: openSUSE Product: openSUSE 11.2 Version: RC 2 Platform: Other OS/Version: Other Status: NEW Severity: Major Priority: P5 - None Component: WebYaST AssignedTo: jreidinger@novell.com ReportedBy: kkaempf@novell.com QAContact: qa@suse.de CC: mschmidkunz@novell.com Found By: Development Every dialog should have one, and only one "ok" or "next" button. "save" is questionable. Having multiple "save" buttons a usability bug. -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=555221 Klaus Kämpf <kkaempf@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|P5 - None |P2 - High AssignedTo|jreidinger@novell.com |jsuchome@novell.com Target Milestone|--- |Final -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=555221 User jsuchome@novell.com added comment http://bugzilla.novell.com/show_bug.cgi?id=555221#c1 Jiří Suchomel <jsuchome@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO CC| |lslezak@novell.com Info Provider| |kkaempf@novell.com --- Comment #1 from Jiří Suchomel <jsuchome@novell.com> 2009-11-13 03:50:40 MST --- Is it? These buttons are intentional, so you can save forwarding email without a need to retyping your root password. -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=555221 User lslezak@novell.com added comment http://bugzilla.novell.com/show_bug.cgi?id=555221#c2 --- Comment #2 from Ladislav Slezak <lslezak@novell.com> 2009-11-13 04:49:01 MST --- See https://bugzilla.novell.com/show_bug.cgi?id=546872#c1 for more details. But right now the passwords fields are empty so we could save the password only when it's not empty. -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=555221 User jsuchome@novell.com added comment http://bugzilla.novell.com/show_bug.cgi?id=555221#c3 --- Comment #3 from Jiří Suchomel <jsuchome@novell.com> 2009-11-13 04:55:58 MST --- (In reply to comment #2)
But right now the passwords fields are empty so we could save the password only when it's not empty.
Wouldn't it be more confusing? Imagine you push main submit button and only see that the settings were written, but you don't know that actually password was not written. (Now it would not be possible anyway, as the password is required if you click password or main submit button) I still like current approach the most. -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=555221 User kkaempf@novell.com added comment http://bugzilla.novell.com/show_bug.cgi?id=555221#c4 Klaus Kämpf <kkaempf@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |NEW Info Provider|kkaempf@novell.com | --- Comment #4 from Klaus Kämpf <kkaempf@novell.com> 2009-11-13 05:04:46 MST --- (In reply to comment #1)
Is it?
These buttons are intentional, so you can save forwarding email without a need to retyping your root password.
This usecase should better be solved with separate dialogs -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=555221 User kkaempf@novell.com added comment http://bugzilla.novell.com/show_bug.cgi?id=555221#c5 Klaus Kämpf <kkaempf@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jsuchome@novell.com AssignedTo|jsuchome@novell.com |mschmidkunz@novell.com --- Comment #5 from Klaus Kämpf <kkaempf@novell.com> 2009-11-13 05:05:48 MST --- => usability -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=555221 User jsuchome@novell.com added comment http://bugzilla.novell.com/show_bug.cgi?id=555221#c6 --- Comment #6 from Jiří Suchomel <jsuchome@novell.com> 2009-11-13 05:17:01 MST --- Well, I'm against having a dialog with just a one entry (email address), IMHO we'd solve one usability bug with another one.... -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=555221#c7 --- Comment #7 from Martin Schmidkunz <mschmidkunz@novell.com> 2009-11-17 11:55:25 UTC --- *** Bug 546872 has been marked as a duplicate of this bug. *** http://bugzilla.novell.com/show_bug.cgi?id=546872 -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=555221#c8 --- Comment #8 from Martin Schmidkunz <mschmidkunz@novell.com> 2009-11-17 12:41:43 UTC --- Concerning users expectations the user just sees a form and the concept of a form (as it is also used in other modules) is to gather information and then to have one possibility to "save"/"cancel" the entries. Having 3 save options is just confusing: * if I press save next to the password dialog and cancel in the main navigation is my password saved? * if I press save in the main navigation and not in the forward field are my changes saved? ..
From the user`s perspective I would also expect, that if an entry exists, it is shown to me me (e.g. with placeholders/stars).
So, my suggestion would be ----------------------------------------------------------- [Icon] System Administrator Settings Password [*********] Forward mail to [ ] Cancel or [Save] -------------------------------------------------------------- "Confirm password" would be displayed if the user made some changes to the password. So, if the password is required for the change of forward mail account it could be sent as it is stored in the form. What do you think? -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=555221#c9 --- Comment #9 from Jiří Suchomel <jsuchome@novell.com> 2009-11-18 07:19:58 UTC --- (In reply to comment #8)
What do you think?
I think we should not change it now, maybe after the initial release. -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=555221#c10 --- Comment #10 from Jiří Suchomel <jsuchome@novell.com> 2009-11-18 07:23:27 UTC --- (In reply to comment #8)
Concerning users expectations the user just sees a form and the concept of a form (as it is also used in other modules) is to gather information and then to have one possibility to "save"/"cancel" the entries. Having 3 save options is just confusing: * if I press save next to the password dialog and cancel in the main navigation is my password saved?
Yes.
* if I press save in the main navigation and not in the forward field are my changes saved?
Yes.
"Confirm password" would be displayed if the user made some changes to the password.
So, if the password is required for the change of forward mail account it could be sent as it is stored in the form.
The password is not required for the forward mail change: and thats exactly why introduced these extra buttons, so you may change the mail and do not touch existing password.I -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=555221#c11 Jiří Suchomel <jsuchome@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO Info Provider| |kkaempf@novell.com --- Comment #11 from Jiří Suchomel <jsuchome@novell.com> 2009-11-18 09:56:13 UTC --- However, this bug has quite a high priority: do you want to remove the Save buttons at all? My main concern with this is in comment 3. Without the extra buttons, user would need to enter (new) root password any time. If this were better than current approach, I could try to remove the buttons, but I'm not sure if I can properly test the solution today. -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=555221#c12 Klaus Kämpf <kkaempf@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |NEW Info Provider|kkaempf@novell.com | --- Comment #12 from Klaus Kämpf <kkaempf@novell.com> 2009-11-18 10:33:02 UTC --- For now, lets leave the buttons but give them proper labels, i.e "Save password", "Save mail" and "Save all" -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=555221#c13 Josef Reidinger <jreidinger@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jreidinger@novell.com --- Comment #13 from Josef Reidinger <jreidinger@novell.com> 2009-11-18 12:58:58 UTC --- I think from user point of view is the best solution have one save and have checkboxes which says if user want to change value (could be also invisible). And if user change value in input, then check checkbox as user want to change value. -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=555221#c14 --- Comment #14 from Jiří Suchomel <jsuchome@novell.com> 2009-11-18 13:03:33 UTC --- yast2-webclient-0.0.13: Save Pasword and Save Mail buttons. No Save All, because that one is generic button not handled inside the plugin. I'm not closing the bug, so we can find better solution for next time. -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=555221#c Jiří Suchomel <jsuchome@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|P2 - High |P3 - Medium -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=555221#c15 --- Comment #15 from Klaus Kämpf <kkaempf@novell.com> 2009-11-18 13:15:03 UTC --- Agreed, thanks ! -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=555221#c16 Klaus Kämpf <kkaempf@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|P3 - Medium |P4 - Low Target Milestone|Final |Future 11.3 --- Comment #16 from Klaus Kämpf <kkaempf@novell.com> 2009-11-18 13:18:17 UTC --- (In reply to comment #13)
I think from user point of view is the best solution have one save and have checkboxes which says if user want to change value (could be also invisible). And if user change value in input, then check checkbox as user want to change value.
I like this proposal :-) How about not showing checkboxes but "revert" links which only appear if value was changed ? This way, users would be able the 'cancel' the value change ? -> downgrading to p4/future as of comment #14 -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=555221 http://bugzilla.novell.com/show_bug.cgi?id=555221#c Klaus Kämpf <kkaempf@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|Major |Enhancement -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply＠novell.com