Closed Bug 1803538 Opened 1 year ago Closed 1 year ago

Add scrollend event in event breakpoints panel

Categories

(DevTools :: Debugger, task)

task

Tracking

(firefox109 fixed)

RESOLVED FIXED
109 Branch
Tracking Status
firefox109 --- fixed

People

(Reporter: nchevobbe, Assigned: nchevobbe)

Details

Attachments

(1 file)

The scrollend event will be enabled by default in Bug 1803435 , it would be nice to show it in the event breakpoints panel

Depends on: 1803435

(In reply to Nicolas Chevobbe [:nchevobbe] from comment #0)

The scrollend event will be enabled by default in Bug 1803435 , it would be nice to show it in the event breakpoints panel

Thanks for creating this! Are we able to add this to the panel before the preference is enabled by default? I've never interacted with the breakpoints panel before, so I'm just curious what happens in the case that apz.scrollend-event.content.enabled were to be false (scrollend are no longer fired to content, but still fired to the browser chrome).

(In reply to Dan Robertson (:dlrobertson) from comment #1)

(In reply to Nicolas Chevobbe [:nchevobbe] from comment #0)

The scrollend event will be enabled by default in Bug 1803435 , it would be nice to show it in the event breakpoints panel

Thanks for creating this! Are we able to add this to the panel before the preference is enabled by default? I've never interacted with the breakpoints panel before, so I'm just curious what happens in the case that apz.scrollend-event.content.enabled were to be false (scrollend are no longer fired to content, but still fired to the browser chrome).

I guess we would only show the item on the list if the pref is enabled so it's not confusing for people.
Which makes me realize, we don't need to wait for the feature to be enabled and can start looking into it right now

Assignee: nobody → nchevobbe
Status: NEW → ASSIGNED
No longer depends on: 1803435

The event is only supported when apz.scrollend-event.content.enabled is set to
true, so we only show it in the debugger when it's supported.

A test is added to ensure the event breakpoint does work as expected.

Pushed by nchevobbe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2e7a46447741
[devtools] Add scrollend in debugger event breakpoint panel. r=bomsy.
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 109 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: