Closed Bug 761481 Opened 13 years ago Closed 13 years ago

GCLI help output does not display the first time

Categories

(DevTools :: General, defect, P1)

14 Branch
x86
macOS
defect

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 16

People

(Reporter: dangoor, Assigned: jwalker)

References

Details

Attachments

(1 file, 1 obsolete file)

STR: 1. With Developer Toolbar closed, press the keyboard shortcut to open the toolbar 2. type "help" and hit enter 3. note the lack of output 4. type "help" and hit enter again 5. ahh, there it is! expected: output at step 3 This is in the 2012-06-04 Nightly on Mac.
Assignee: jwalker → nobody
Thank you, I meant to file this last weekend but it slipped my mind.
Priority: -- → P1
Summary: [gcli] help output does not display the first time → GCLI help output does not display the first time
Target Milestone: --- → Firefox 16
Hey Neil - I wonder if you could help with a panel problem. 2 things to add to Kevin's STR: - the pref to enable the toolbar is: devtools.toolbar.enabled=true - this happens on Mac/Linux but not Windows If you open browser/devtools/shared/DeveloperToolbar.jsm, line 354, you can see the call to openPopup, it is called at the right time on all OSes. Relevant to the discussion is the _resize() function just below show(), although from adding debug to that function, it looks like sane values are being used for the height/width (i.e. 265x275) and those numbers are the same whether the output is displayed or not. Thanks.
The panel is first resized to one size, then some left margin is being added to it, then the panel is resized to accomodate this margin. <panel xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul" id="gcli-tooltip" class="gcli-panel" noautofocus="true" noautohide="true"><iframe xmlns="http://www.w3.org/1999/xhtml" height="0" id="gcli-tooltip-frame" src="chrome://browser/content/devtools/gclitooltip.xhtml" flex="1" style="margin-left: 10px;"></iframe></panel> Maybe you mean to add the margin on the panel itself, or in the iframe's content? Is the margin or its size dependent on the size of the panel? If not, it should be applied before opening the panel.
Er, that comment was meant for bug 760450
Why is the panel being resized three times when being opened?
(In reply to Neil Deakin from comment #5) > Why is the panel being resized three times when being opened? Twice? this._panel.ownerDocument.defaultView.setTimeout(function() { this._resize(); }.bind(this), 0); this._panel.openPopup(this._input, "before_start", 0, 0, false, false, null); this._resize(); this._input.focus(); Displaying the panel causes it to re-flow, which can change the size it should be, so we need to resize the iframe after the panel has displayed.
The panel is also being resized at: else if (aEvent.type == "resize") { this.outputPanel._resize(); }
(In reply to Neil Deakin from comment #7) > The panel is also being resized at: > > else if (aEvent.type == "resize") { > this.outputPanel._resize(); > } Ah, right that's so that when the window is resized, we ensure the output panel is the same width as the input area.
Blocks: 745773
Neil - I'd really like to get this fixed, can I get you any more info to help work out what's wrong?
I think I tested commenting out the resize I mentioned in comment 7 and the size worked fine. You might want to see if there's a way to skip that in this case. Possibly by seeing if the window really did change size.
Blocks: 768150
Attached patch Upload 1 (obsolete) — Splinter Review
Assignee: nobody → jwalker
Status: NEW → ASSIGNED
Attachment #636424 - Flags: review?(dcamp)
Attachment #636424 - Flags: review?(dcamp) → review+
Attached patch Upload 2Splinter Review
The previous version of this patch was done in somewhat of a hurry, this is a slightly better version of the same thing.
Attachment #636424 - Attachment is obsolete: true
Attachment #637439 - Flags: review?(dcamp)
Attachment #637439 - Flags: review?(dcamp) → review+
Whiteboard: [fixed-in-fx-team]
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
This issue has returned.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Actually it's not a problem with this bug.
Status: REOPENED → RESOLVED
Closed: 13 years ago13 years ago
Resolution: --- → FIXED
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: