Closed
Bug 1490086
Opened 6 years ago
Closed 6 years ago
Parenthesis do not match an unmatched closing parenthesis
Categories
(DevTools :: Console, defect, P1)
Tracking
(firefox64 fixed)
RESOLVED
FIXED
Firefox 64
Tracking | Status | |
---|---|---|
firefox64 | --- | fixed |
People
(Reporter: gregtatum, Assigned: bgrins)
Details
(Whiteboard: [boogaloo-mvp])
Attachments
(1 file)
Steps to reproduce:
* Open up console.
* Type `10 - 5 / 2`
* Move text cursor to the left and insert a closing parenthesis after the 5
* `10 - 5) / 2`
* Move text cursor to the beginning, and add an opening parenthesis.
Expected behavior:
* The text to be `(10 - 5) / 2`
Actual behavior:
* The text is `()10 - 5) / 2`.
Comment 1•6 years ago
|
||
So, this is because of the `autoCloseEnabled` config of codeMirror.
It is driven by the "Autoclose brackets" option in Settings panel (under "Editor Preferences").
We could either: turn it off in the console (Chrome does not seem to have such thing), or fix the underlying issue (not not close if there's a single closing brackets after) on codeMirror.
Not sure what would be the best to do.
The easiest would definitely to not have autoclose in the console. So maybe we can do that for the moment, and then later fix the issue and respect the user choice from the settings ?
Priority: -- → P2
Whiteboard: [boogaloo-reserve]
Updated•6 years ago
|
Priority: P2 → P3
Reporter | ||
Comment 2•6 years ago
|
||
Personally I really don't like auto bracketing behavior, as it breaks my typing habits, so I would be a fan of removing it all together. I know there is much more that goes into that decision though.
Assignee | ||
Comment 3•6 years ago
|
||
(In reply to Greg Tatum [:gregtatum] from comment #2)
> Personally I really don't like auto bracketing behavior, as it breaks my
> typing habits, so I would be a fan of removing it all together. I know there
> is much more that goes into that decision though.
Yeah, I originally thought we should respect the global option. But given STR like Comment 0, and hitting this inadvertently through normal console use I'd lean more towards forcing it off for the console.
Assignee | ||
Comment 4•6 years ago
|
||
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → bgrinstead
Status: NEW → ASSIGNED
Updated•6 years ago
|
Priority: P3 → P1
Whiteboard: [boogaloo-reserve] → [boogaloo-mvp]
Comment 5•6 years ago
|
||
Comment on attachment 9009266 [details]
Bug 1490086 - Disable autoCloseBrackets feature for console input;r=nchevobbe
Nicolas Chevobbe [:nchevobbe] has approved the revision.
Attachment #9009266 -
Flags: review+
Pushed by bgrinstead@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/416b3f8e0883
Disable autoCloseBrackets feature for console input;r=nchevobbe
Comment 7•6 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
You need to log in
before you can comment on or make changes to this bug.
Description
•