Closed Bug 1329308 Opened 8 years ago Closed 7 years ago

Reverse current "use experimental ui" user preference and make it instead "use legacy bug ui" or something similar

Categories

(bugzilla.mozilla.org :: User Interface, defect)

Production
defect
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: dkl, Assigned: seban)

Details

Attachments

(1 file)

Once we turn on the bug modal form as default, IMO it will no longer be experimental and considered the standard bug edit form going forward. Therefore I think it would be less confusing if we reverse the user preference and allow people to turn 'on' the legacy form instead of turning 'off' the new form. dkl
Summary: Reverse current 'use experimental ui" user prefernce and make it instead "use legacy bug ui" or something similar → Reverse current "use experimental ui" user preference and make it instead "use legacy bug ui" or something similar
Attached patch 1329308_1.patch (deleted) — Splinter Review
I am going to remove this a blocker. The way it is implemented, it would be better to push this patch and run it after users have been given a week or so to decide if they are going to turn off the modal UI or leave it on. Then we can do the conversion of the settings to the new setting. dkl
Attachment #8828174 - Flags: review?(dylan)
Removing as a blocker as we will need to run this a week or so after the modal UI becomes the default. dkl
No longer blocks: 1150541
Comment on attachment 8828174 [details] [diff] [review] 1329308_1.patch r=dylan with two nits: 1) If I've explicitly turned off the new UI with the old option, this commit overrides that choice. Is that intentional? I support it, but I wanted to make sure it was an intentional decision. 2) I think the preference should be named ui_legacy_show_bug, or ui_legacy_something, but not "interface" as the i in UI means "interface".
Attachment #8828174 - Flags: review?(dylan) → review+
You can probably assign this to sebastin, if you like
Flags: needinfo?(dkl)
(In reply to Dylan Hardison [:dylan] (he/him) from comment #4) > You can probably assign this to sebastin, if you like Sorry for letting this fall through the cracks. Assigning to Sebastin. dkl
Assignee: dkl → sebastinssanty
Flags: needinfo?(dkl)
I don't see a reason to complete this. The current setting is fine, and eventually the setting goes away.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
Component: User Interface: Modal → User Interface
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: