[BeOS] Cannot open menus after workspace change.

RESOLVED FIXED

Status

Core Graveyard
Widget: BeOS
RESOLVED FIXED
12 years ago
2 years ago

People

(Reporter: Sergei Dolgov, Assigned: Sergei Dolgov)

Tracking

({fixed1.8})

Trunk
x86
BeOS
fixed1.8

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

12 years ago
Open some menu from menu bar.
Bring browser to another workspace.
Try to open menu. No luck.

Reason - menus seems to be rather hidden than destroyed in "closing". So they
stay in old workspace.

Remedy - when unhiding popups, bring those always to current workspace.
It is also may be good idea to bring there non-popup dependent windows, like
Preferences, History etc, but it seems much more complicated task. We should
consult Unix people, who are also dealing with multiple workspaces.
(Assignee)

Comment 1

12 years ago
Created attachment 197687 [details] [diff] [review]
Patch.

Bringing popup to current workspace before Show()
Attachment #197687 - Flags: review?(thesuckiestemail)

Comment 2

12 years ago
Comment on attachment 197687 [details] [diff] [review]
Patch.

r=thesuckiestemail@yahoo.se
Attachment #197687 - Flags: review?(thesuckiestemail) → review+

Updated

12 years ago
Blocks: 266252
(Assignee)

Comment 3

12 years ago
Landed in trunk.
/cvsroot/mozilla/widget/src/beos/nsWindow.cpp,v  <--  nsWindow.cpp
new revision: 1.96; previous revision: 1.95 
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED

Comment 4

12 years ago
Comment on attachment 197687 [details] [diff] [review]
Patch.

BeOS only change. won't affect any other platforms. Requesting approval for the
1.8 branch.
Attachment #197687 - Flags: approval1.8b5?
(Assignee)

Comment 5

12 years ago
Comment on attachment 197687 [details] [diff] [review]
Patch.

extra }, sorry. Build bustage.
Trunk fixed
new revision: 1.97; previous revision: 1.96 

submitting fixed patch for 1.8* soon
Attachment #197687 - Attachment is obsolete: true
(Assignee)

Comment 6

12 years ago
Created attachment 197727 [details] [diff] [review]
patch

removed extra }
Nielx, please test it again before 1.8* commit (if patch applies cleanly and
builds ok).
I'm very sorry

Updated

12 years ago
Attachment #197687 - Flags: approval1.8b5? → approval1.8b5+
Checking in widget/src/beos/nsWindow.cpp;
/cvsroot/mozilla/widget/src/beos/nsWindow.cpp,v  <--  nsWindow.cpp
new revision: 1.91.4.5; previous revision: 1.91.4.4
done

Comment 8

12 years ago
did this land on the branch? If so, please add the fixed1.8 keyword. If not,
then it didn't make 1.8b5 so please request approval for rc1 if you think it's
warranted.

Updated

12 years ago
Keywords: fixed1.8
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.