`View' > `Show/Hide' menu items can't override window.open flags




18 years ago
a year ago


(Reporter: mpt, Unassigned)


(Depends on: 1 bug, Blocks: 1 bug)

Dependency tree / graph

Firefox Tracking Flags

(Not tracked)


(Whiteboard: [p-ie/mac], URL)


(1 attachment, 1 obsolete attachment)



18 years ago
Build: 2001040408, Mac OS 9.1.

To reproduce:
*   Open a Navigator window, and go to <http://numberfinder.com/>.
*   Perform a search for anything.
*   In the window which comes up, use the `View' > `Toolbars' submenu to turn
    the toolbars back on.

What should happen:
*   The items in the `View' > `Toolbars' menu should be unchecked, and checking
    them should turn the toolbars back on.

What actually happens:
*   The items in the `View' > `Toolbars' menu are checked, and unchecking them
    and re-checking them has no effect.


18 years ago
Blocks: 26353
The window that comes up for me doesn't have a menu bar, so I can't use the 
View menu to re-show the toolbar.

Doh! Of course. I see. MacOS. Yes. Right.


Comment 3

18 years ago
That means pchen, I think.
OS: other → Mac System 9.x
Hardware: Other → Macintosh
C, d'you see this?
QA Contact: sairuh → claudius

Comment 5

18 years ago
yup, in 2001041008 builds, but I wasn't really surprised. Isn't this exactly what the author intended?
It wouldn't even be possible on another platform. Maybe the 'toolbar' item should just be greyed out?

regardless, reassigning to appropriate people.
Assignee: ben → pinkerton
Component: XP Apps: GUI Features → XP Toolkit/Widgets: Menus
QA Contact: claudius → jrgm
-> pchen
Assignee: pinkerton → pchen
Component: XP Toolkit/Widgets: Menus → XP Apps: GUI Features

Comment 7

18 years ago
I don't see what purpose disabling the items would serve, other than annoying 
people. The author got what he/she asked for -- a window without chrome. Now 
when I decide to turn the chrome on again (because it's *my* Web browser, not 
the Web author's), the toolbar items don't do anything. I can turn the toolbars 
back on in Internet Explorer, why not in Mozilla?
Whiteboard: [parity-ie/mac]

Comment 8

18 years ago
-> me, if Paul doesn't mind...
Assignee: pchen → blakeross

Comment 9

18 years ago
Sorry, I didn't realize this was Mac-only when I took it (I think I have 
another bug somewhere on these items not working on other platforms when the 
popup has a menubar).  Back to Paul, but e-mail me if you're willing to send me 
a mac and I'll respond with my address.
Assignee: blakeross → pchen
Priority: -- → P1
Target Milestone: --- → mozilla0.9.1
Keywords: nsbeta1-


18 years ago
Target Milestone: mozilla0.9.1 → Future

Comment 10

18 years ago
I have reproduced problem. Click on side tab to "close" the tab, then go to View
-> Toolbar and uncheck the item.  It won't come back or show. Happens in Mail as
well as browser. 


18 years ago
Blocks: 103711
->default assignee
Assignee: pchen → blaker
QA Contact: jrgm → sairuh
Target Milestone: Future → ---
->XP Apps default assignee
Assignee: blaker → trudelle
Component: XP Apps: GUI Features → XP Apps
sorry, this was supposed to be XPApps: GUI Features
Assignee: trudelle → blaker
Component: XP Apps → XP Apps: GUI Features


17 years ago
Target Milestone: --- → Future

Comment 14

17 years ago
Also broken on Windows:
javascript:window.open("", "", "menubar=yes"); void 0
OS: Mac System 9.x → All
Hardware: Macintosh → All
*** Bug 75742 has been marked as a duplicate of this bug. ***


17 years ago
Summary: `View' > `Toolbars' items don't work in chromeless window → `View' > `Show/Hide' menu items can't override window.open flags
Whiteboard: [parity-ie/mac] → [p-ie/mac]

Comment 16

17 years ago

3.  var chromehidden=document.documentElement.getAttribute("chromehidden");
    var morechrome=chromehidden.replace(/\btoolbar\b/g," ");
    if (chromehidden!=morechrome) 
        do the usual thing

Comment 17

17 years ago
taking... i got some lines from timeless which i converted into a real fix *eg*,
i will attach it as soon as my cygwin setup is finished... (or tomorrow,
whichever one comes first)
Assignee: blaker → rossi
Posted patch Proposed patch (obsolete) — Splinter Review
Note that this patch does not do anything about the checkmarks on the menu
because they are persisted so changing them to reflect the window.open flags
would impact on new windows. Nor does this patch fix the sidebar.
Posted patch Updated patchSplinter Review
Attachment #84058 - Attachment is obsolete: true
Comment on attachment 84631 [details] [diff] [review]
Updated patch

I have talked to jst about this a while ago.  We don't want to fix it this way,
since it will not update the window flags it needs to.	There is going to be
some c++ stuff that I'll code up hopefully sometime soon that will take care of
this and some other problems with window features as well.
Attachment #84631 - Flags: needs-work+
I've been looking for a placeholder for some of this stuff.  This bug will do
just nicely.
Assignee: rossi → caillon
Component: XP Apps: GUI Features → DOM Level 0
Depends on: 55820, 155660
Keywords: patch, polish, review, ui
Priority: P1 → P3
QA Contact: sairuh → desale
Assignee: caillon → general
QA Contact: desale → ian


15 years ago
Blocks: 244412

Comment 22

15 years ago
*** Bug 251329 has been marked as a duplicate of this bug. ***


14 years ago
No longer blocks: 244412

Comment 23

14 years ago
*** Bug 244412 has been marked as a duplicate of this bug. ***
Assignee: general → nobody
QA Contact: ian → general
Bulk priority change, per :mdaly
Priority: P3 → P5
You need to log in before you can comment on or make changes to this bug.