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 55160 - Drag & Drop of folders will lock the GUI
Summary: Drag & Drop of folders will lock the GUI
Status: RESOLVED FIXED
Alias: None
Product: platform
Classification: Unclassified
Component: Explorer (show other bugs)
Version: 3.x
Hardware: PC Windows ME/2000
: P3 blocker (vote)
Assignee: Jaroslav Tulach
URL:
Keywords: API_REVIEW_FAST, PERFORMANCE
: 27744 81334 (view as bug list)
Depends on:
Blocks:
 
Reported: 2005-02-18 01:24 UTC by adrianlewty
Modified: 2008-12-22 23:45 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
Patch returning back revision 1.6 plus adding use of Progress API (3.28 KB, patch)
2006-01-11 17:32 UTC, Jaroslav Tulach
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description adrianlewty 2005-02-18 01:24:31 UTC
Drag & Drop a folder will cause the GUI to lock 
up (gray screen) until the copying of the 
folder/files has been completed.

This is confusing for users for when a folder 
contains 1000 items or more, the GUI may be 
locked up for more than 10 minutes and so the 
user ends up killing the application thinking it 
has crashed.

I would expect the same functionality as when you 
do a copy & paste which still leaves the GUI 
responsive while the copy & paste is occurring in 
the background.

What can happen too is that the GUI will forever 
remain locked. Checking via Windows Explorer will 
show that no files at all have been copied to the 
dropped target. The GUI then has to be killed. I 
think this has happened when you have 1500 or 
more items in the folder.

I've reproduced this in 3.6, 4.0 and 4.1.
Comment 1 Petr Nejedly 2005-02-22 11:52:28 UTC
This is not nice.
Comment 2 Petr Nejedly 2005-03-15 13:43:20 UTC
PasteType implementation in datasystems performs paste synchronously. It doesn't
block UI for Paste (Ctrl+V) because PasteAction is asynchronous and thus runs in
a request processor.
I'll add replanning to RP into the datasystems PasteType in case it is called
from inside AWT.
Comment 3 Petr Nejedly 2005-03-15 14:33:07 UTC
Fixed.
openide/loaders/src/org/openide/loaders/DataTransferSupport.java,v
Comment 4 Jaroslav Tulach 2005-03-24 10:42:04 UTC
cvs ci -m "#56854: Reverting fix for issue 55160 - e.g. drop is done
synchronously. Bad, but better than broken refactoring. I'll find a different
way to move the drop out of AWT event thread"
Comment 5 Jaroslav Tulach 2005-03-24 10:43:35 UTC
But I do not think I'll make it for 4.1. Sorry.
Comment 6 Jaroslav Tulach 2005-11-09 13:56:24 UTC
Delaying for next version, I've been told not to change UI (e.g. introduce the 
progress indicator, which is needed). 
 
Comment 7 Jaroslav Tulach 2006-01-11 17:32:47 UTC
Created attachment 28353 [details]
Patch returning back revision 1.6 plus adding use of Progress API
Comment 8 Jaroslav Tulach 2006-01-11 17:33:58 UTC
I want to make the org.openide.loaders module depend on progress API. Any 
objections from reviewers? 
Comment 9 Jaroslav Tulach 2006-01-17 10:30:14 UTC
No comments so far, I am going to add the dependency. 
Comment 10 Jaroslav Tulach 2006-01-18 08:49:53 UTC
"#55160: Using progress API for Dnd to not block AWT" 
Checking in openide/loaders/nbproject/project.xml; 
/cvs/openide/loaders/nbproject/project.xml,v  <--  project.xml 
new revision: 1.12; previous revision: 1.11 
done 
Checking in openide/loaders/src/org/openide/loaders/DataTransferSupport.java; 
/cvs/openide/loaders/src/org/openide/loaders/DataTransferSupport.java,v  <--  
DataTransferSupport.java 
new revision: 1.9; previous revision: 1.8 
done 
Checking in ide/golden/deps.txt; 
/cvs/ide/golden/deps.txt,v  <--  deps.txt 
new revision: 1.275; previous revision: 1.274 
 
Comment 11 Jaroslav Tulach 2006-04-21 14:11:47 UTC
*** Issue 27744 has been marked as a duplicate of this issue. ***
Comment 12 Jan Becicka 2006-08-04 17:05:20 UTC
*** Issue 81334 has been marked as a duplicate of this issue. ***