Closed
Bug 1488566
Opened 6 years ago
Closed 6 years ago
Unable to leave RDM, "No localization found for [responsive.reloadConditions.label]"
Categories
(DevTools :: Responsive Design Mode, defect, P2)
DevTools
Responsive Design Mode
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 1488114
People
(Reporter: jdescottes, Assigned: jdescottes)
References
Details
Attachments
(1 obsolete file)
STRs: - clean profile - open any website - check "Reload when user agent is changed" - change the user agent - try to stop RDM Impossible to leave RDM, user agent of the browser is messed up after restarting etc... Browser console logs Error: "No localization found for [responsive.reloadConditions.label]" Regression from https://hg.mozilla.org/mozilla-central/rev/81d81c2f9098 Bug 1467572 part 9
Assignee | ||
Comment 1•6 years ago
|
||
Note that the feature itself is useless in its current state, because it displays a banner on the content page which is being reloaded. We should pass command: true to showNotification in order to show the banner in the toolbox (and therefore persist the message after the reload). However this will need to wait after Bug 1429421
Comment 2•6 years ago
|
||
I actually fixed this in Bug 1488114 as well.
Comment 3•6 years ago
|
||
Comment on attachment 9006368 [details] Bug 1488566 - Update reload notification message to stop using missing string;r=gl,flod Francesco Lodolo [:flod] has approved the revision.
Attachment #9006368 -
Flags: review+
Assignee | ||
Comment 4•6 years ago
|
||
gl: this needs an uplift to beta, are you confident we can uplift the refactor?
Flags: needinfo?(gl)
Comment 5•6 years ago
|
||
Comment on attachment 9006368 [details] Bug 1488566 - Update reload notification message to stop using missing string;r=gl,flod Gabriel [:gl] (ΦωΦ) has approved the revision.
Attachment #9006368 -
Flags: review+
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Flags: needinfo?(gl)
Assignee | ||
Comment 6•6 years ago
|
||
I assumed you would pull the duplicated changes out of the other bug, now this can't land on central anymore. Let's move everything to your other bug.
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
Updated•6 years ago
|
Attachment #9006368 -
Attachment is obsolete: true
You need to log in
before you can comment on or make changes to this bug.
Description
•