Closed
Bug 2043714
Opened 2 months ago
Closed 1 month ago
DoH custom select UI forgets that it is in custom mode on reload
Categories
(Firefox :: Settings UI, defect, P2)
Firefox
Settings UI
Tracking
()
RESOLVED
FIXED
153 Branch
| Tracking | Status | |
|---|---|---|
| firefox-esr115 | --- | unaffected |
| firefox-esr140 | --- | unaffected |
| firefox151 | --- | wontfix |
| firefox152 | --- | wontfix |
| firefox153 | --- | fixed |
People
(Reporter: bvandersloot, Assigned: bvandersloot)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
The select reports to be the first element in the list when reloaded with a custom url. This is because the get() happens before the getControlConfig(). Found via Connect: https://connect.mozilla.org/t5/discussions/the-redesign-of-settings-is-now-enabled-by-default-in-nightly/m-p/126794/highlight/true#M50165
| Assignee | ||
Comment 1•2 months ago
|
||
Updated•2 months ago
|
Assignee: nobody → bvandersloot
Status: NEW → ASSIGNED
Comment 2•2 months ago
|
||
Set release status flags based on info from the regressing bug 1971428
status-firefox151:
--- → affected
status-firefox152:
--- → affected
status-firefox153:
--- → affected
status-firefox-esr140:
--- → unaffected
Updated•2 months ago
|
Severity: -- → S3
Priority: -- → P2
Updated•2 months ago
|
Flags: in-testsuite+
Pushed by bvandersloot@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/dd27bcf85335
https://hg.mozilla.org/integration/autoland/rev/6443ac01ab37
Fix DoH custom select memory - r=manuel,Gijs
Comment 4•1 month ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 1 month ago
Resolution: --- → FIXED
Target Milestone: --- → 153 Branch
Comment 5•1 month ago
|
||
The patch landed in nightly and beta is affected.
:bvandersloot, is this bug important enough to require an uplift?
- If yes, please nominate the patch for beta approval.
- See https://wiki.mozilla.org/Release_Management/Requesting_an_Uplift for documentation on how to request an uplift.
- If no, please set
status-firefox152towontfix.
For more information, please visit BugBot documentation.
Flags: needinfo?(bvandersloot)
| Assignee | ||
Comment 6•1 month ago
|
||
We don't need the uplift, so let's not take it to lighten the burden.
Flags: needinfo?(bvandersloot)
Updated•1 month ago
|
Updated•15 days ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•