Closed
Bug 1718713
Opened 4 years ago
Closed 4 years ago
Create dummy custom element in new screenshot.js
Categories
(Firefox :: Screenshots, enhancement, P2)
Firefox
Screenshots
Tracking
()
RESOLVED
FIXED
92 Branch
Tracking | Status | |
---|---|---|
firefox92 | --- | fixed |
People
(Reporter: emmamalysz, Assigned: kajalsah107, NeedInfo)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
This bug can track part 3 of the project. Let's try and hook up a simple custom element in screenshots.js (maybe just a single div) and make sure we show that on the screen if screenshots.browser.component.enabled
is true.
In the places that we've implemented the SCREENSHOT_BROWSER_COMPONENT
checks, we should be injecting this custom element into the html when the pref we added is true.
Reporter | ||
Updated•4 years ago
|
Severity: -- → S4
Priority: -- → P2
Pushed by emalysz@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/57c50b132961
Creates dummy custom element in new screenshot. r=emalysz
Comment 3•4 years ago
|
||
Backed out for causing eslint failures.
Backout link: https://hg.mozilla.org/integration/autoland/rev/156a909b6a8de01ee2bb1a0723d4d2aa7d3f342e
Flags: needinfo?(kajalsah107)
Pushed by emalysz@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/54c1bca9d887
Creates dummy custom element in new screenshot. r=emalysz
Comment 5•4 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox92:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 92 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•