Closed
Bug 773565
Opened 12 years ago
Closed 12 years ago
GCLI Autocomplete goes wild when boolean params are used in a group
Categories
(DevTools :: Console, defect, P1)
DevTools
Console
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 17
People
(Reporter: miker, Assigned: jwalker)
References
Details
(Whiteboard: [gclicommands][fixed-in-fx-team])
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
miker
:
review+
|
Details | Diff | Splinter Review |
GCLI Autocomplete goes wild when boolean params are used in a group:
1. Install the patch from bug 769560
2. Type the following:
jsb http://code.jquery.com/jquery-1.7.2.min.js --jslintHappy
3. Press space and you get the completion for braceStyle.
4. Add --braceStyle and choose collapse from the popup and it overwrites braceStyle
Assignee | ||
Comment 1•12 years ago
|
||
Triage, filter on TEABAGS
Component: Developer Tools → Developer Tools: Console
Priority: -- → P1
Target Milestone: --- → Firefox 17
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → jwalker
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•12 years ago
|
||
Attachment #648452 -
Flags: review?(dcamp)
Updated•12 years ago
|
Attachment #648452 -
Flags: review?(dcamp) → review+
Reporter | ||
Comment 3•12 years ago
|
||
Joe said that if we add the following to OutputPanel.prototype.update = function OP_update():
if (this.displayedOutput.data.completed) {
// Hide progress
} else {
// Show progress
}
Here is a complete(ish) progress implementation from elsewhere:
OutputView.prototype.changed = function(ev) {
var document = this.elems.rowout.ownerDocument;
var duration = this.outputData.duration != null ?
'completed in ' + (this.outputData.duration / 1000) + ' sec ' :
'';
duration = document.createTextNode(duration);
this.elems.duration.appendChild(duration);
if (this.outputData.completed) {
this.elems.prompt.classList.add('gcli-row-complete');
}
if (this.outputData.error) {
this.elems.prompt.classList.add('gcli-row-error');
}
this.outputData.toDom(this.elems.rowout);
// We need to see the output of the latest command entered
// Certain browsers have a bug such that scrollHeight is too small
// when content does not fill the client area of the element
var scrollHeight = Math.max(this.outputTerminal.element.scrollHeight,
this.outputTerminal.element.clientHeight);
this.outputTerminal.element.scrollTop =
scrollHeight - this.outputTerminal.element.clientHeight;
this.elems.throb.style.display = this.outputData.completed ? 'none' : 'block';
};
Assignee: jwalker → mratcliffe
Reporter | ||
Comment 4•12 years ago
|
||
Ignore my previous comment, wrong bug.
Assignee: mratcliffe → jwalker
Assignee | ||
Comment 5•12 years ago
|
||
rebase and update tests to suite in gcli
Attachment #648452 -
Attachment is obsolete: true
Reporter | ||
Comment 6•12 years ago
|
||
Comment on attachment 654968 [details] [diff] [review]
v2
Review of attachment 654968 [details] [diff] [review]:
-----------------------------------------------------------------
::: browser/devtools/commandline/CmdJsb.jsm
@@ +118,5 @@
> + if (xhr.status == 200 || xhr.status == 0) {
> + let browserDoc = context.environment.chromeDocument;
> + let browserWindow = browserDoc.defaultView;
> + let browser = browserWindow.gBrowser;
> +
Trailing space
Attachment #654968 -
Flags: review+
Comment 7•12 years ago
|
||
Backed out in:
https://hg.mozilla.org/integration/fx-team/rev/f4596ef17eed
Assignee | ||
Comment 8•12 years ago
|
||
Whiteboard: [gclicommands] → [gclicommands][fixed-in-fx-team]
Comment 9•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 10•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/c80d0e010be3 was the relanded changeset.
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•