This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 17691 - Dock into debugger window doesn't work; must make visible
Summary: Dock into debugger window doesn't work; must make visible
Status: CLOSED FIXED
Alias: None
Product: debugger
Classification: Unclassified
Component: Code (show other bugs)
Version: 3.x
Hardware: All All
: P2 blocker (vote)
Assignee: Jan Jancura
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2001-11-15 00:09 UTC by Torbjorn Norbye
Modified: 2001-12-18 10:56 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Torbjorn Norbye 2001-11-15 00:09:08 UTC
The following patch is needed to DebuggerWindow:
Index: src/org/netbeans/modules/debugger/support/nodes/DebuggerWindow.java
===================================================================
RCS file:
/cvs/debuggercore/src/org/netbeans/modules/debugger/support/nodes/DebuggerWindow.java,v
retrieving revision 1.16
diff -c -r1.16 DebuggerWindow.java
*** src/org/netbeans/modules/debugger/support/nodes/DebuggerWindow.java	10 Nov 2001 02:55:32 -0000	1.16
--- src/org/netbeans/modules/debugger/support/nodes/DebuggerWindow.java	15 Nov 2001 00:00:27 -0000
***************
*** 465,470 ****
--- 465,477 ----
                  initSpecialViews (v, c);
              }*/
                  
+ 	    // Ensure that the component is not hidden. This happens when
+ 	    // components are closed as part of a dock-into-debugging window
+ 	    // operation.
+             if (!c.isVisible()) {
+ 		c.setVisible(true);
+ 	    }
+ 	    
              if (c instanceof ToolbarView) {
                  relToolbar = false;
                  ((ToolbarView) c).setOrientation (


That's because sometimes moving a component away from a tabbed window
and into the debuggerwindow causes the component to be called with
setVisible(false) (the tab manager ui is doing this) so by similar reasoning
the new manager, the DebuggerWindow, should make the views visible.

Note that this will be a no-op in most cases since generally views haven't
been made hidden by some prior window operation - thus it's low risk
for you. And in the cases where isVisible() is false, you -want- the above
setVisible(true).
Comment 1 Jan Jancura 2001-11-15 16:36:05 UTC
in main trunk
Comment 2 Jan Stola 2001-12-18 10:56:48 UTC
Closed.