You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Select the reason "The package has a bug" from the dropdown. As expected, the other form elements will be disabled.
Press the browser's previous page and next page buttons.
Result: On returning to the Report Abuse page, the other form elements will no longer be disabled. You can submit the form with this reason.
Expected: This should not be possible. I've filed a separate but to unify our handling of invalid reasons, but barring that, we should try to catch this scenario on return and re-disable the other elements on the page if "The package has a bug" is the currently-selected reason.
Contribution Notes
Contributed by @mgroves - This should be noted in the Change Log/Release Notes.
The text was updated successfully, but these errors were encountered:
I'll take a quick look in I7, but this is ripe for being deferred or pushed to Up For Grabs :). This is really just a way to reduce noise, not necessarily eliminate it.
Result: On returning to the Report Abuse page, the other form elements will no longer be disabled. You can submit the form with this reason.
Expected: This should not be possible. I've filed a separate but to unify our handling of invalid reasons, but barring that, we should try to catch this scenario on return and re-disable the other elements on the page if "The package has a bug" is the currently-selected reason.
Contribution Notes
Contributed by @mgroves - This should be noted in the Change Log/Release Notes.
The text was updated successfully, but these errors were encountered: