File -> Open Location nonfunctional when the Location Toolbar is hidden

RESOLVED FIXED in Firefox1.5

Status

()

Firefox
Menus
P3
normal
RESOLVED FIXED
13 years ago
11 years ago

People

(Reporter: Miriam Frenay, Assigned: mano)

Tracking

unspecified
Firefox1.5
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

13 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.6) Gecko/20050225 Firefox/1.0.1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.6) Gecko/20050225 Firefox/1.0.1

When the Location Toolbar is hidden (e.g. in a javascript popup) the File ->
Open Location menu item does not display the Open Web Location dialog.

Reproducible: Always

Steps to Reproduce:
1. Set the dom.disable_window_open_feature.menubar preference to true
2. Open a javascript link that opens a new window with the location toolbar hidden
3. Select the File -> Open Location menu item

Actual Results:  
Nothing

Expected Results:  
Display the Open Web Location dialog
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee: firefox → bugs.mano
OS: Windows XP → All
Priority: -- → P3
Hardware: PC → All
Target Milestone: --- → Firefox1.1
Status: NEW → ASSIGNED
Where does "display: none" ever get set on the toolbar? Isn't it the result of
having "moz-collapsed" or "chromehidden" attributes, or something else that you
can use instead of getting the computed style?
Gavin, the chromehidden attribute "produces" display:none, see relevant rules in
xul.css (and no, it doesn't affect the collapsed property)
(In reply to comment #3)
> Gavin, the chromehidden attribute "produces" display:none

Right, so why don't you use |hasAttribute("chromehidden")| then? That seems
cleaner to me, and probably is less expensive than computing the style.

Am I missing something?
(In reply to comment #4)
> (In reply to comment #3)
> > Gavin, the chromehidden attribute "produces" display:none
> 
> Right, so why don't you use |hasAttribute("chromehidden")|

chromehidden doesn't store the location bar status alone, you'll need to pass it
in a regexp.

> and probably is less expensive than computing the style.

no, it isn't.

Updated

13 years ago
Attachment #177850 - Flags: review?(mconnor) → review+
Checking in browser.js;
/cvsroot/mozilla/browser/base/content/browser.js,v  <--  browser.js
new revision: 1.402; previous revision: 1.401
done
Status: ASSIGNED → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED

Updated

11 years ago
QA Contact: bugzilla → menus
You need to log in before you can comment on or make changes to this bug.