Closed Bug 1563850 Opened 6 years ago Closed 3 years ago

Add test for markup view event tooltip "Open In Debugger" when script was pretty-printed

Categories

(DevTools :: Inspector, enhancement, P2)

Product:
DevTools
Component:
Inspector
Type:
enhancement

Tracking

(Not tracked)

Status:
RESOLVED WORKSFORME

People

(Reporter: ehsan.akhgari, Unassigned)

References

(Blocks 1 open bug)

Details

When clicking on "Open In Debugger" inside Inspector on a list of event listeners that points to minified code, where I have already put the source code in pretty print code, it's really annoying that Firefox keeps opening the original source view, since it typically focuses line 1, which isn't useful for setting a breakpoint on, reading the code, etc.

It would be nice to use the existing pretty print view if one is available.

This sounds like a reasonable request. We did this recently for minified CSS files opened in the Style Editor (Bug 724505).
It would be nice to have the behaviour consistent for opening sources in the Debugger too.

Priority: -- → P2
See Also: → 1262803

Same issue occurs in the debugger when clicking the go-away button in the hover pop-up:

Not sure if this the same issue.

Sorry, Bugzilla seems to have eaten the markdown image of the screenshot of the button so I am posting a link.

Severity: normal → S3

I tested it and it does work fine, but I don't think we have a test for it (or adding a test case to browser_markup_view-source.js)

Summary: When clicking on "Open In Debugger" in Inspector, open the code in the pretty print view if one is available → Add test for markup view event tooltip "Open In Debugger" when script was pretty-printed

Test was added in Bug 1010150

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.