Closed
Bug 1795334
Opened 2 years ago
Closed 1 year ago
Add a pref that opens the existing migration.xhtml document in a gDialogBox window modal
Categories
(Firefox :: Migration, task, P3)
Firefox
Migration
Tracking
()
RESOLVED
FIXED
109 Branch
Tracking | Status | |
---|---|---|
firefox109 | --- | fixed |
People
(Reporter: mconley, Assigned: steven100695)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
We're going to slowly start moving the migration dialog into a modern window modal, rather than an OS modal window.
This project has a few steps:
- Create a pref for a new migration dialog and default it to false.
- Update the code here to read that pref, and if it's true, to open the dialog using the opener's
gDialogBox
instead. Here's an example of doing something like this: https://searchfox.org/mozilla-central/rev/a64647a2125cf3d334451051491fef6772e8eb57/browser/components/places/PlacesUIUtils.sys.mjs#543
where in this context, aParentWindow
should be aOpener
.
We'll then want to build out a very basic test for this variation.
It is not expected that the new variation in the new window modal will actually fully work. We just want it to show up there.
Reporter | ||
Updated•2 years ago
|
Assignee: nobody → steven100695
Updated•2 years ago
|
Severity: -- → N/A
Priority: -- → P3
Updated•1 year ago
|
Attachment #9303143 -
Attachment description: Bug 1795334 - Add a pref that opens the existing migration.xhtml document in a gDialogBox window modal. r?mconley. → Bug 1795334 - Add a pref that opens the existing migration.xhtml document in a tab dialog box modal. r?mconley!.
Pushed by mconley@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/eb918188305a Add a pref that opens the existing migration.xhtml document in a tab dialog box modal. r=mconley.
Comment 3•1 year ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 1 year ago
status-firefox109:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 109 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•