Closed Bug 1819355 Opened 2 years ago Closed 2 years ago

addLineClass/removeLineClass are using an undocumented param value

Categories

(DevTools :: Debugger, defect, P3)

defect

Tracking

(firefox112 fixed)

RESOLVED FIXED
112 Branch
Tracking Status
firefox112 --- fixed

People

(Reporter: nchevobbe, Assigned: marlenegomes1, Mentored)

Details

Attachments

(1 file)

In a lot of places we are using "wrapClass" as the second argument: https://searchfox.org/mozilla-central/search?q=%22wrapClass%22&path=devtools%2Fclient%2F&case=true&regexp=false

But the documentation does not mention this value:

can be one of "text" , "background" […], "gutter" […], or "wrap" […]

Mentor: nchevobbe
Severity: -- → S3
Priority: -- → P3

can I work on this?

Thanks for helping, the bug is now assigned to you

Assignee: nobody → marlenegomes1
Status: NEW → ASSIGNED
Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/1045676996d2 Replace wrapClass by wrap in addLineClass/removeLineClass. r=nchevobbe.
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: