Closed Bug 45899 Opened 24 years ago Closed 24 years ago

Save dialog doesn't appear when sidebar has focus

Categories

(Core :: DOM: Editor, defect, P2)

Product:
Core
Component:
DOM: Editor
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: sujay, Assigned: cmanske)

References

Details

(Whiteboard: [nsbeta3+][p:2]) 

using 7/19 build of netscape

1) launch netscape
2) launch composer
3) enter some text
4) give focus in the sidebar window
5) File | Save As

notice the Save dialog doesn't come up...

now give focus back to the editor

do File | Save As again...it comes up this time.

all platforms.
*** Bug 45897 has been marked as a duplicate of this bug. ***
Wrong summary?
Fixed the weird summary.
Summary: Pasting yields "P" char → Save dialog doesn't appear when sidebar has focus
Anthony can you see if this is an easy fix, this may end up being some kind of 
focus notification issue that maybe saari will need to help with
Assignee: beppe → anthonyd
Keywords: correctness
Target Milestone: --- → M19
Status: NEW → ASSIGNED
ok, so the real problem with this bug is that File Save|File Save As should be 
ghosted when document is not focused.

Anthony

*** Bug 41179 has been marked as a duplicate of this bug. ***
*** Bug 45920 has been marked as a duplicate of this bug. ***
This affects menus in mail compose, etc.  See dups.
Nominating for nsbeta3.
Keywords: nsbeta3
this shold be fixed for beta3
Whiteboard: nsbeta3+
Target Milestone: M19 → M18
Whiteboard: nsbeta3+ → [nsbeta3+]
setting priority in status whiteboard
Priority: P3 → P2
Whiteboard: [nsbeta3+] → [nsbeta3+][p:2]
Okay, so after discussing this a bit with saari, and others, it appears that the 
correct behaviour is to ghost inappropriate menu items if the document is not 
focused.  Until I hear otherwise, that is the solution I am going to persue.

Anthony
Depends on: 49376
Charlie has fixed this, reassigning to him so he can close this after he checks 
in.
Assignee: anthonyd → cmanske
Status: ASSIGNED → NEW
checked in 8/22. Adding controller to main window (fix for 41810) also fixed 
this.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
verified in 8/24 build.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.