768277 - When clicking Add Exception, the "Permanently store this exception" option check in the checkbox is delayed

Status: NEW

(Firefox :: Security, defect, P5)

Description

[Tanvi Vyas[:tanvi]]

When you first click the "Add Exception" button, the checkbox appears to be unpopulated (because the checkbox hasn't been populated yet).  A user goes to the "Confirm Security Exception" button, and waits for it to be clickable (it is greyed out to begin with).  The second it becomes clickable is the same second that the "Permanently store this exception" box gets checked.  If a user quickly presses the Confirm button (right after it is clickable), they will think they are only temporarily storing the exception, when in fact they realize after the click that this is permanent because the permanent checkbox just checked itself.

This is terrible user experience, because then the user has to figure out how to remove this permanent exception.

Creating this as a clone to bug 535439, which discussed whether the box should or should not be checked bye default.

+++ This bug was initially created as a clone of Bug #536197 +++

Comment 1

[Tanvi Vyas[:tanvi]]

Any thoughts on who might take this on?  Or which team/person I should cc on this bug?  Thanks!

Comment 2

[Mike Hommey [:glandium]]

It seems to me this isn't true anymore.

Comment 3

[Tanvi Vyas[:tanvi]]

On nightly, I still see a delay before the box shows up as checked.  It appears unchecked at first, and a user could hit submit before realizing that the box checks itself after a short delay.

Comment 4

[Brian Smith (:briansmith, :bsmith, use NEEDINFO?)]

Bug 795771 is about making the UI for desktop and mobile products more consistent and overall more usable. Please CC yourself with that if you are interested in the eventual design.

I would accept a patch for this bug if somebody wrote one. Presumably, the "right" fix is to enable the OK button only after we've computed the final state of the UI (i.e. after we've checked the checkbox).

Comment 5

[Tanvi Vyas[:tanvi]]

Attachment: Greyed out cert override showing the Permanently store this exception box unchecked

(In reply to Brian Smith (:briansmith, was :[e-mail]) from comment #4)
> I would accept a patch for this bug if somebody wrote one. Presumably, the
> "right" fix is to enable the OK button only after we've computed the final
> state of the UI (i.e. after we've checked the checkbox).

I'm not convinced that this is the right fix.  On a slow computer, I can hover over the "Confirm Security Exception" button and click it as soon as it is not greyed out, thinking that I am only adding the exception for this one time.  At about the same time I click on it, I will notice that the "Permanently store this exception" option is checked all of a sudden, but it will be too late for me to do anything about it.

Why don't we just have it checked from the beginning (during the greyed out period and afterwards).

Comment 6

[Sylvestre Ledru [:Sylvestre]]

Decreasing the priority as no update for the last 2 years on this bug.
See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage 
about the priority meaning.