Closed Bug 1897922 Opened 29 days ago Closed 18 days ago

Use the EditorView.domEventHandlers for DOM events instead of attaching to the line content extension

Categories

(DevTools :: General, task)

Product:
DevTools
Component:
General
Type:
task

Tracking

(firefox128 fixed)

Status:
RESOLVED FIXED
Milestone:
128 Branch
Tracking Flags:
Tracking Status
firefox128 --- fixed

People

(Reporter: bomsy, Assigned: bomsy)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Bug 1897922 - [devtools] Manage DOM event using EditorView.domEventHandlers r=#devtools-reviewers
48 bytes, text/x-phabricator-request
Details | Review

Currently the DOM events are attached to the line content extension (which also manages line content markers). Let the DOM events be handled by the built-in extension EditorView.domEventHandlers.

Assignee: nobody → hmanilla
Status: NEW → ASSIGNED
Pushed by hmanilla@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4dbf4414edde
[devtools] Manage DOM event using EditorView.domEventHandlers r=devtools-reviewers,nchevobbe

Backed out for causing dt failures @ devtools/client/shared/test/browser_telemetry_toolboxtabs_webconsole.js

Backout link: https://hg.mozilla.org/integration/autoland/rev/234eac4ce3091fb7382a2b2c2fd8875fd3591c75

Push with failures

Failure log ->

Flags: needinfo?(hmanilla)

Fixed the issues.

Flags: needinfo?(hmanilla)
Pushed by hmanilla@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e70ed7a0708f
[devtools] Manage DOM event using EditorView.domEventHandlers r=devtools-reviewers,nchevobbe

https://hg.mozilla.org/mozilla-central/rev/e70ed7a0708f

Status: ASSIGNED → RESOLVED
Closed: 18 days ago
status-firefox128: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 128 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: