Closed Bug 1955235 Opened 1 year ago Closed 1 year ago

Remove inline style from CodeMirror addon/search

Categories

(DevTools :: General, task)

task

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";.

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.

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.
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 138 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: