Closed Bug 3675 Opened 26 years ago Closed 26 years ago

[PP]Opening the bookmarks window replaces the Navigator window

Categories

(Core Graveyard :: Tracking, defect, P2)

x86
Linux
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: mikepinkerton, Assigned: danm.moz)

Details

Launch apprunner and use the "Edit Bookmarks" menu item to bring up a new bookmarks window. Move the window so that you can see the original browser window, and you'll see that it too is a bookmarks window. This is on mac On linux, a second window isn't opened. The main window becomes the bookmarks window.
This works fine on win32.
Assignee: trudelle → danm
Priority: P3 → P2
Target Milestone: M3
reassigning to danm as p2 for m3
it's not just bookmarks. choosing "messenger" from the apps menu does the same thing.
Ok, I was wrong. On MacOS, it opens two bookmarks windows (one covering your old window). The old browser window is still there. On linux, however, there really is only one window.
OS: Mac System 8.5 → Linux
Hardware: All → PC
Summary: Opening the bookmarks window makes all windows the bookmarks window → Opening the bookmarks window replaces the Navigator window
The mac problem (which was actually all menu events were being dispatched twice, yes, it is really was totally different bug) is now fixed. Marking this as a linux bug and changing the title to reflect it.
Status: NEW → ASSIGNED
Current status: I have a fix on my local build which seems to work. I'm in an overnight discussion with GTK Pavlov about it -- he has some worries about what I'm doing. So the problem is known, a (perhaps inappropriate but functional) solution is also known. One way or another, it'll be all better in time for M3.
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → REMIND
This is bandaided to work for now. The real problem is a miscommunication between the three platforms about what it means to have a top-level window with a parent. Parents for top-level windows have been disabled, so the bookmarks window will now show up in a separate window. We need a real solution to the actual problem; hopefully next week. (But a note for the eager-to-test: you generally can't open the bookmarks window on Linux. See bug 3716.)
Status: RESOLVED → REOPENED
Target Milestone: M3 → M4
re-opening and moving to M4. As noted above, the symptom of the problem has been fixed and remains fixed. We'd like a real fix for the future. Also note that bug 3716 mentioned above seems to be a problem no longer today.
Target Milestone: M4 → M5
Summary: Opening the bookmarks window replaces the Navigator window → [PP]Opening the bookmarks window replaces the Navigator window
Target Milestone: M5 → M6
QA Contact: 3853 → 4130
updating QA Contact.
Target Milestone: M6 → M7
to what does this refer to now? Is 'Edit Bookmarks' supposed to be Bookmarks|Manage Bookmarks? The behavior is different now isn't it? This have been changed to work properly, no?
Yes, he says conversationally, we're talking about Bookmarks/Manage Bookmarks. It appears to behave correctly, but this is accomplished by obscuring another problem. Window parentage is currently disabled. That's a nice thing whose absence will soon be missed. If we re-enable it, this bookmarks problem comes back. It's still unfixed. It might more appropriately be a new and different bug.
Target Milestone: M7 → M9
Moving all Apprunner bugs past and present to Other component temporarily whilst don and I set correct component. Apprunner component will be deleted/retired shortly.
Target Milestone: M9 → M11
Status: REOPENED → RESOLVED
Closed: 26 years ago26 years ago
Resolution: REMIND → FIXED
This one's so old I don't remember what the problem was. I do know that the symptom has been fixed, and the underlying problem that I wanted to leave this bug open to remind me about, that being that top-level windows couldn't have parents, has just now been fixed. Parent information is no longer being thrown away and everybody's happy now. Closing.
I've been waiting for this actually, it's stained on my whiteboard now. Is the 'parents o' top level windows' stuff checked in (and turned on)? If so when cuz, that way I can make sure the Bookmarks aren't broken and this very old bug will finally go away.
Status: RESOLVED → VERIFIED
Marking VERIFIED for 1999081609 builds
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.