Patch #11057
Confirm admin flag change on a separate page
Status: | New | Start date: | ||
---|---|---|---|---|
Priority: | Normal | Due date: | ||
Assignee: | - | % Done: | 0% | |
Category: | Administration | |||
Target version: | - |
Description
Having admin flag on the general tab in user's profile makes it
way too easy to flip the flag accidentally.
Attached patch adds a separate page to flip the flag.
History
#1
Updated by Jean-Philippe Lang almost 10 years ago
It's a bit overkill IMO. Why not adding a simple javascript confirmation message when changing the checkbox state?
#2
Updated by Etienne Massip almost 10 years ago
Why not simply move it in its own "Administrator" fieldset?
Not sure it should remain in "Information" anyway, this is a security related concept, not a general information?
#3
Updated by Alex Shulgin almost 10 years ago
Jean-Philippe Lang wrote:
It's a bit overkill IMO. Why not adding a simple javascript confirmation message when changing the checkbox state?
I disagree: a simple Yes/No popup is too easy to overlook or accept accidentally.
Etienne Massip wrote:
Why not simply move it in its own "Administrator" fieldset?
Not sure it should remain in "Information" anyway, this is a security related concept, not a general information?
Well, maybe that's a way to go. Want me to produce an adjusted patch with a separate tab?
#4
Updated by Alex Shulgin almost 10 years ago
Alex Shulgin wrote:
Etienne Massip wrote:
Why not simply move it in its own "Administrator" fieldset?
Not sure it should remain in "Information" anyway, this is a security related concept, not a general information?
Well, maybe that's a way to go. Want me to produce an adjusted patch with a separate tab?
Attached is a reworked version with the admin tab instead of a separate page.