Closed Bug 1757581 Opened 4 years ago Closed 4 years ago

Triggering view-source when the console input has focus should open page source not do nothing

Categories

(DevTools :: Console, defect, P3)

defect

Tracking

(firefox100 verified)

VERIFIED FIXED
100 Branch
Tracking Status
firefox100 --- verified

People

(Reporter: glob, Assigned: nchevobbe)

Details

Attachments

(1 file)

Steps to reproduce

  1. Navigate to any web page
  2. Open dev tools Console and ensure it has focus
  3. Press "View Source" keyboard shortcut (eg. cmd+U on macOS)

Expected results

  • Source for the current web page is opened in a tab

Actual result

  • Menu item flashes
  • Nothing else 🙁

The severity field is not set for this bug.
:nchevobbe, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(nchevobbe)

Note that this is only true when the console input has the focus. If you put the cursor in the "Filter Output" element and hit Cmd + U, it does work.
It seems that Ctrl+U is captured by CodeMirror for it undoSelection command (https://codemirror.net/doc/manual.html#commands), but
we can probably drop that and do the same thing we did than for Cmd/Ctrl+F (devtools/client/webconsole/components/Input/JSTerm.js#500-501 )

Assignee: nobody → nchevobbe
Severity: -- → S3
Status: NEW → ASSIGNED
Flags: needinfo?(nchevobbe)
Priority: -- → P3
Summary: Triggering view-source when the console has focus should open page source not do nothing → Triggering view-source when the console input has focus should open page source not do nothing

The shortcut is already used to open view-source and we want to let it through.
CodeMirror is using Ctrl+U for its undoSelection command, but it doesn't seem
like an important feature, so we can simply drop it.

Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ced98831523c [devtools] Don't handle Ctrl/Cmd+U in JSTerm. r=jdescottes.
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 100 Branch
QA Whiteboard: [qa-100b-p2]

I managed to reproduce this on a 2022-03-01 Nightly build(20220301215224) on macOS 11. Verified as fixed on Firefox Firefox 100.0(20220428192727) and Nightly 101.0a1(20220501190841) on macOS 11, Win10 64-bits and Ubuntu 20.04.

Status: RESOLVED → VERIFIED
QA Whiteboard: [qa-100b-p2]

🎉 ty!

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: