Created attachment 258296 [details] [diff] [review] v1 Window opened by toOpenWindowByType have a chromehidden="location" attribute by default, which hides the location bar.
Attachment #258296 - Flags: review?(dbaron)
Comment on attachment 258296 [details] [diff] [review] v1 This is fine with me, although maybe it's better to drop the class="chromeclass-location"? I don't know. Maybe Gavin does. But I don't think it matters much.
Attachment #258296 - Flags: review?(dbaron) → review+
I think it would be better to: 1) remove class="chromeclass-location" from the layoutdebug textbox. I suspect it's only there because someone copy/pasted it from a browser window. Opening a layout debug window without the location bar makes no sense. 2) File a bug on making toOpenWindowByType() use a better default feature set (I think "all,dialog=no" makes more sense, and matches the xpfe behavior after bug 136901).
Created attachment 258339 [details] [diff] [review] v2 I opened bug 373690 for the toOpenWindowByType enhancement.
Attachment #258339 - Flags: review?(gavin.sharp) → review+
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
Whiteboard: [checkin needed]
You need to log in before you can comment on or make changes to this bug.