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 43909 - [ui] ProgressPanel dialog is modal
Summary: [ui] ProgressPanel dialog is modal
Status: RESOLVED FIXED
Alias: None
Product: java
Classification: Unclassified
Component: Unsupported (show other bugs)
Version: 4.x
Hardware: All All
: P2 blocker (vote)
Assignee: Jan Becicka
URL:
Keywords:
: 47415 48419 (view as bug list)
Depends on:
Blocks:
 
Reported: 2004-05-27 04:59 UTC by _ tball
Modified: 2007-09-26 09:14 UTC (History)
2 users (show)

See Also:
Issue Type: ENHANCEMENT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description _ tball 2004-05-27 04:59:28 UTC
When the ProgressPanel is displayed, its dialog
container is intentionally set modal (line 133). 
This prevents any other interaction with the IDE,
even tasks that don't require any JMI support. 
This dialog should be modeless, so as not to block
the user.
Comment 1 Jan Becicka 2004-05-27 08:13:55 UTC
Dialog is modal in order to disallow user to modify source during
searching.
Comment 2 Martin Matula 2004-05-27 09:04:37 UTC
This is as designed. Making the dialog non-modal would bring in too
many problems for only a little value assuming that users usually
start a Java IDE to edit Java source code - this requires JMI since
opening a file in the editor involves many calls to Java source
hierarchy (to populate the drop-down listbox of features in the
editor's toolbar, to display overriden annotations, to figure out
feature positions for code folding, etc.).
AFAIK the scanning is modal even in the currently best
refactoring-enabled IDEs.
Comment 3 Jan Becicka 2004-09-14 10:14:35 UTC
*** Issue 48419 has been marked as a duplicate of this issue. ***
Comment 4 Jan Becicka 2004-09-14 10:16:25 UTC
Reopening as enhancement for future releases.
Comment 5 Jan Becicka 2004-11-04 11:17:55 UTC

*** This issue has been marked as a duplicate of 45078 ***
Comment 6 Jan Becicka 2005-02-10 14:09:08 UTC
Let's take a look at it.
Comment 7 Jan Becicka 2005-02-15 09:19:49 UTC
There is a new switch
-J-Dnetbeans.javacore.background_scanning=true
which forces IDE to do scanning on background.
This switch is experimental (not official) and bugs related to this
switch can be closed as WONTFIX (for now). Anyway we appreciate your
feedback (preferably add your experiences and opinions as a comments
to this issue).

Known issues: code completion, code folds and all stuff requiring java
model access are postponed till scanning is finished.
Comment 8 _ gtzabari 2005-02-15 15:12:45 UTC
I don't know how you handle things from a usability point of view, but
if the user tries invoking code-complete and other operations that
require scanning to be over I would expect you'd pop up a dialog that
says something along the lines of:

"The command you have invoked requires Netbeans to scan your
classpath. This operation is currently ongoing in the background. You
can either wait for this operation to complete or abort your last
command. This dialog will automatically close when the scanning
operation is complete.

ETA: <estimated time left for scanning operation>

[ Cancel ]
"

That's not perfect UI, but you get my point :P

Anyway, I think that would be neat.
Comment 9 Jan Becicka 2005-02-15 15:37:34 UTC
Thanks for your feedback, Gili.
I don't know, whether you tried background_scanning or not, but
progress of scanning *is* shown, and code completion shows message
"please wait" until scanning operation is complete.
But I must admit, that your dialog is more descriptive.

It is not and probably will not be possible to [Cancel] scanning task.
Comment 10 _ tball 2005-02-15 21:38:12 UTC
I like it -- ship it!  Perhaps those actions that depend on scanning
being finished could be disabled until it's available.  It's fine the
way it is, however.

One minor bug report:  on Linux using JDK 1.5, the toolbar's "chrome"
isn't restored after the progress bar is removed.  That may be a Swing
bug, however.
Comment 11 Jan Becicka 2005-02-28 09:48:36 UTC
> One minor bug report:  on Linux using JDK 1.5, the toolbar's "chrome"
> isn't restored after the progress bar is removed.  That may be a Swing
> bug, however.

Should be fixed now.
Comment 12 Jan Becicka 2005-04-04 12:30:34 UTC
*** Issue 47415 has been marked as a duplicate of this issue. ***
Comment 13 _ gtzabari 2005-05-01 16:48:48 UTC
Has anyone had problems with this background scanning feature? I've been using
it for a few days now with absolutely no problems. Any chance of us enabling it
by default?
Comment 14 Martin Matula 2005-05-02 08:28:45 UTC
It will be enabled by default as soon as 4.1 is released.
Comment 15 Jan Becicka 2005-05-12 14:16:25 UTC
Fixed.

Checking in java/javacore/nbproject/project.xml;
/cvs/java/javacore/nbproject/project.xml,v  <--  project.xml
new revision: 1.10; previous revision: 1.9
done
Checking in java/javacore/src/org/netbeans/modules/javacore/Bundle.properties;
/cvs/java/javacore/src/org/netbeans/modules/javacore/Bundle.properties,v  <-- 
Bundle.properties
new revision: 1.16; previous revision: 1.15
done
Checking in java/javacore/src/org/netbeans/modules/javacore/JMManager.java;
/cvs/java/javacore/src/org/netbeans/modules/javacore/JMManager.java,v  <-- 
JMManager.java
new revision: 1.108; previous revision: 1.107
done
Removing java/javacore/src/org/netbeans/modules/javacore/ProgressAction.java;
/cvs/java/javacore/src/org/netbeans/modules/javacore/ProgressAction.java,v  <--
 ProgressAction.java
new revision: delete; previous revision: 1.2
done
RCS file:
/cvs/java/javacore/src/org/netbeans/modules/javacore/ProgressDisplayer.java,v
done
Checking in java/javacore/src/org/netbeans/modules/javacore/ProgressDisplayer.java;
/cvs/java/javacore/src/org/netbeans/modules/javacore/ProgressDisplayer.java,v 
<--  ProgressDisplayer.java
initial revision: 1.1
done
Removing java/javacore/src/org/netbeans/modules/javacore/ProgressPanel.java;
/cvs/java/javacore/src/org/netbeans/modules/javacore/ProgressPanel.java,v  <-- 
ProgressPanel.java
new revision: delete; previous revision: 1.16
done
Removing java/javacore/src/org/netbeans/modules/javacore/ProgressPanel2.java;
/cvs/java/javacore/src/org/netbeans/modules/javacore/ProgressPanel2.java,v  <--
 ProgressPanel2.java
new revision: delete; previous revision: 1.5
done
Checking in
java/javacore/src/org/netbeans/modules/javacore/scanning/FileScanner.java;
/cvs/java/javacore/src/org/netbeans/modules/javacore/scanning/FileScanner.java,v
 <--  FileScanner.java
new revision: 1.25; previous revision: 1.24
done
Checking in ide/golden/cluster-deps.txt;
/cvs/ide/golden/cluster-deps.txt,v  <--  cluster-deps.txt
new revision: 1.13; previous revision: 1.12
done
Checking in ide/golden/deps.txt;
/cvs/ide/golden/deps.txt,v  <--  deps.txt
new revision: 1.67; previous revision: 1.66
done
Checking in ide/test/qa-functional/src/validation/IDEValidation.java;
/cvs/ide/test/qa-functional/src/validation/IDEValidation.java,v  <-- 
IDEValidation.java
new revision: 1.113; previous revision: 1.112
done
Comment 16 Quality Engineering 2007-09-20 09:58:24 UTC
Reorganization of java component