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 23223

Summary: File|Open -- mount needed filesystems without any dialogs
Product: utilities Reporter: Jan Benway <benway>
Component: CodeAssignee: issues@utilities <issues>
Status: RESOLVED FIXED    
Severity: blocker CC: issues, jglick
Priority: P3 Keywords: UI
Version: 3.x   
Hardware: All   
OS: All   
Issue Type: ENHANCEMENT Exception Reporter:
Attachments: Proposed fix.

Description Jan Benway 2002-05-07 18:19:41 UTC
When the user does file|open to bring in a file in an unmounted filesystem, they get a dialog 
box confirming that the IDE should mount the filesystem at the package root level. This dialog 
did not help new users, as it is intended to do, in a recent usability test.

For the report see: http://ui.netbeans.org/usability/Mar_20_02/index.html#mount
(to be checked in soon).

I propose removing the confirmation dialog box and the dialog box allowing the user to change 
the mount point, and just mount the needed filesystem in the background, without asking the 
user.
Comment 1 Jesse Glick 2002-05-08 18:22:16 UTC
Utilities module implements this.

What if the selected file is not a Java source file? In that case a
different dialog is displayed - no guess is possible. What should be
done there?
Comment 2 Jan Benway 2002-05-08 18:30:59 UTC
When I open a text file, no dialog is displayed, it just mounts the
directory it is in. So it already follows the behavior described here.

When I open a zip file, it brings up a dialog asking if I want to
mount it, yes or no. If I say no, it does nothing. I think we could
eliminate this dialog, and just mount the zip (why else would I ask to
open it?)

What other file types should I try to see other dialogs?
Comment 3 Jesse Glick 2002-05-08 18:34:51 UTC
Re. opening a text file - OK, I thought there was a dialog for that,
but maybe it got deleted long ago.

Re. ZIP files - agreed.

Re. other file types - I think that's it.
Comment 4 _ lkramolis 2002-05-14 19:57:21 UTC
Created attachment 5739 [details]
Proposed fix.
Comment 5 _ lkramolis 2002-05-14 19:59:35 UTC
Fixed in main trunk, please review. Thanks.
Comment 6 Jesse Glick 2002-05-14 20:21:47 UTC
Patch looks reasonable to me.

BTW about the patch itself - (1) when commenting out code using a
bundle key, it is a good idea to comment out the bundle key too, so it
is clear it is not in active use; (2) /* ... */ block comments are
sometimes nicer for commenting something out in a patch, as CVS treats
them as just adding two lines, rather than replacing every line in the
block with a //-commented-out version - this makes the patch text look
clearer, and will make cvs annotate work better in the future.