Closed
Bug 917777
Opened 11 years ago
Closed 11 years ago
Control + A for selecting everything is not working
Categories
(DevTools :: Console, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 28
People
(Reporter: leo, Assigned: paul)
Details
(Whiteboard: [good first bug][lang=js][mentor=msucan][good first verify])
Attachments
(1 file, 2 obsolete files)
1.57 KB,
patch
|
msucan
:
review+
|
Details | Diff | Splinter Review |
You cannot select the entire content of the console input field by pressing the normal shortcus Control + A. It simply is doing nothing. But thats a problem, because sometimes you just want to clear the entire input field.
Comment 1•11 years ago
|
||
Thank you for the bug report. Ctrl-A/E are working as expected on Linux. We had this discussion previously in bug 623749. (leaving open until the next web console bugs triage when we will decide if we want to change this behavior or not)
Reporter | ||
Comment 2•11 years ago
|
||
In my opinion this is a bug. Because for jumping to the first/last position of the input field you already habe the "pos1" and "end" key on your keyboard. Also the field down there is not a reall command line, so the comparison to the linux terminal is not correct. In every input element CTRL+A ist select all. Why try to change it in this case?
Comment 3•11 years ago
|
||
The team discussed this and consensus was to keep the existing behavior. FWIW you can use Ctrl-Shift-A to Select All on Linux.
Status: UNCONFIRMED → RESOLVED
Closed: 11 years ago
Resolution: --- → WONTFIX
Assignee | ||
Comment 4•11 years ago
|
||
I'm going to be the annoying guy here :) Re-opening because this behavior always seriously bugged me. (In reply to Leo Unglaub from comment #2) > In every input element CTRL+A ist select all. Why try to change it in this > case? I agree. Even in other browsers Ctrl-A selects the line. (In reply to Panos Astithas [:past] from comment #3) > The team discussed this and consensus was to keep the existing behavior. Can we re-discuss that?
Status: RESOLVED → REOPENED
Ever confirmed: true
Resolution: WONTFIX → ---
Updated•11 years ago
|
Priority: -- → P2
Comment 5•11 years ago
|
||
let's do this for windows only because those users have a HOME key to get to the beginning of the line.
Whiteboard: [good first bug][lang=js][mentor=msucan]
Assignee | ||
Comment 6•11 years ago
|
||
I want that for Linux :(
Well, everyone can press some version of a Home key... Windows and Linux machines tend to have an explicitly labelled key. For Mac, Apple's extended keyboard has a Home key, or you can press Fn + Left Arrow to get the same thing.
Assignee | ||
Comment 8•11 years ago
|
||
Assignee | ||
Comment 9•11 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=4ef1b1a9bcea
Assignee: nobody → paul
Attachment #8336707 -
Attachment is obsolete: true
Status: REOPENED → ASSIGNED
Attachment #8336709 -
Flags: review?(mihai.sucan)
Comment 10•11 years ago
|
||
Comment on attachment 8336709 [details] [diff] [review] v1 Review of attachment 8336709 [details] [diff] [review]: ----------------------------------------------------------------- Thank you! ::: browser/devtools/webconsole/webconsole.js @@ +3984,5 @@ > // control-a > this.clearCompletion(); > > + if (Services.appinfo.OS != "Darwin") { > + // Allow Select All on Windows and Linux. This is fine, but do we need to keep the code for this case (ctrl-a)? Afaik, ctrl-a is implemented natively on macs. Can you please check?
Attachment #8336709 -
Flags: review?(mihai.sucan) → review+
Assignee | ||
Comment 11•11 years ago
|
||
(In reply to Mihai Sucan [:msucan] from comment #10) > This is fine, but do we need to keep the code for this case (ctrl-a)? Afaik, > ctrl-a is implemented natively on macs. Can you please check? You're right.
Assignee | ||
Comment 12•11 years ago
|
||
Attachment #8336709 -
Attachment is obsolete: true
Attachment #8336735 -
Flags: review?(mihai.sucan)
Comment 13•11 years ago
|
||
Comment on attachment 8336735 [details] [diff] [review] v2 Thanks!
Attachment #8336735 -
Flags: review?(mihai.sucan) → review+
Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Comment 14•11 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/437b05dda7ca
Keywords: checkin-needed
Whiteboard: [good first bug][lang=js][mentor=msucan] → [good first bug][lang=js][mentor=msucan][fixed-in-fx-team]
Comment 15•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/437b05dda7ca
Status: ASSIGNED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Whiteboard: [good first bug][lang=js][mentor=msucan][fixed-in-fx-team] → [good first bug][lang=js][mentor=msucan]
Target Milestone: --- → Firefox 28
Updated•10 years ago
|
Whiteboard: [good first bug][lang=js][mentor=msucan] → [good first bug][lang=js][mentor=msucan][good first verify]
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•