Closed
Bug 1955235
Opened 1 year ago
Closed 1 year ago
Remove inline style from CodeMirror addon/search
Categories
(DevTools :: General, task)
DevTools
General
Tracking
(firefox138 fixed)
RESOLVED
FIXED
138 Branch
| Tracking | Status | |
|---|---|---|
| firefox138 | --- | fixed |
People
(Reporter: tschuster, Assigned: nchevobbe)
References
(Blocks 1 open bug)
Details
Attachments
(1 external file)
The following sets an inline style attribute using JS: https://searchfox.org/mozilla-central/source/devtools/client/shared/sourceeditor/codemirror/addon/search/search.js#234,239,256. They should be replaced with code like elem.style.width = "20em";.
| Assignee | ||
Comment 1•1 year ago
|
||
Instead use the class that are set on the elements to add CSS rules in mozilla.css.
The classes are renamed to make it clear they're for CodeMirror 5.
The style of the element is slightly tweaked to use our theme variables, and we
set proper focus indicator on the search/replace input.
Updated•1 year ago
|
Assignee: nobody → nchevobbe
Status: NEW → ASSIGNED
Pushed by nchevobbe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/edfa3a7c50d9
[devtools] Remove inline style from codemirror search addon. r=devtools-reviewers,ochameau.
Comment 3•1 year ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox138:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 138 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•