After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 377157 - Add DnD files onto archiver files with help of file-roller
Add DnD files onto archiver files with help of file-roller
Status: RESOLVED FIXED
Product: nautilus
Classification: Core
Component: Views: All
2.23.x
Other All
: Normal enhancement
: ---
Assigned To: Nautilus Maintainers
Nautilus Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-11-19 21:39 UTC by Nelson Benitez
Modified: 2008-08-04 23:07 UTC
See Also:
GNOME target: ---
GNOME version: Unversioned Enhancement


Attachments
Patch to add DnD files onto archiver files in nautilus (5.82 KB, patch)
2006-11-19 21:46 UTC, Nelson Benitez
none Details | Review
Updated patch (5.92 KB, patch)
2006-11-25 16:30 UTC, Nelson Benitez
none Details | Review
New version to fix minor things (5.81 KB, patch)
2006-11-25 21:59 UTC, Nelson Benitez
none Details | Review
Now really looks for extension at end of filename (5.81 KB, patch)
2006-11-25 23:16 UTC, Nelson Benitez
none Details | Review
Patch to add DnD files onto archiver files in nautilus based on mime-type (6.96 KB, patch)
2006-12-03 20:28 UTC, Nelson Benitez
needs-work Details | Review
Updated patch to nautilus svn trunk (5.55 KB, patch)
2008-05-27 09:17 UTC, Nelson Benitez
none Details | Review

Description Nelson Benitez 2006-11-19 21:39:38 UTC
Hi, I like this feature that I normally use in windows file manager to quickly add files to zip archives. You just drag some files and drop them onto a zip|rar file and it gets added to it. I'll attach a patch for making this in nautilus in conjunction with file-roller.
Comment 1 Nelson Benitez 2006-11-19 21:46:29 UTC
Created attachment 76868 [details] [review]
Patch to add DnD files onto archiver files in nautilus

I used GDK_ACTION_LINK as GdkDragAction because I tried with GDK_ACTION_PRIVATE and also GDK_ACTION_DEFAULT and neither of them got triggered in drag_data_received_callback even if I added them to gtk_drag_dest_set in nautilus_icon_dnd_init and in nautilus-icon-container.c:motion_notify_event() 

With GDK_ACTION_LINK is working fine.
Comment 2 Nelson Benitez 2006-11-19 21:58:28 UTC
Also see a screencast[1] of this feature (beware is 2.2MB gif file) in which I add some small files and then a 3.3MB file for which file-roller shows a nice progress bar till file gets added... I have to thank file-roller for having nice commandline option for adding files...

[1] http://supertux.cenobioracing.com/archivos/nautilus_roller.gif
Comment 3 Nelson Benitez 2006-11-25 16:30:27 UTC
Created attachment 77129 [details] [review]
Updated patch

This new patch adds some improvements from comments on nautilus-list:
 - nautilus_is_archiver_file now checks that the extension is at the end of filename.
 - Use GDK_ACTION_COPY instead of GDK_ACTION_LINK, and it works for me now!
 - The launching code was moved to fm-directory-view so now it is used from both icon view and list view.
Comment 4 Nelson Benitez 2006-11-25 21:59:51 UTC
Created attachment 77141 [details] [review]
New version to fix minor things

Just fix indentation and remove unneeded comment...
Comment 5 Nelson Benitez 2006-11-25 23:16:40 UTC
Created attachment 77147 [details] [review]
Now really looks for extension at end of filename

Now really looks for extension at end of filename.
Comment 6 Nelson Benitez 2006-12-03 20:28:10 UTC
Created attachment 77598 [details] [review]
Patch to add DnD files onto archiver files in nautilus based on mime-type

This new patch just changes nautilus_is_archiver_file() to use mime types instead of file extensions, so it will integrate with nautilus mime type recognition, to show that I made this screencast:
http://supertux.cenobioracing.com/archivos/roller_mime.gif
Comment 7 A. Walton 2008-05-18 14:40:27 UTC
Thanks for your work on this. I'm personally not sure if this is something we want to support or not, but I don't want to see your efforts go to waste here! If you still want this feature, can you make an updated version of it against the GIO system and send it to the Nautilus Mailing List for consideration? I'm setting the patch statuses based on your last two comments (the first being obsolete, the second being "needs-work" w.r.t. GIO-port.) Thanks again.

Helpful links:
http://mail.gnome.org/mailman/listinfo/nautilus-list
http://library.gnome.org/devel/gio/unstable/
Comment 8 Nelson Benitez 2008-05-18 15:01:24 UTC
(In reply to comment #7)
> Thanks for your work on this. I'm personally not sure if this is something we

Ok, I'll work on an updated version of the patch, I hope to have it cooked for the next weekend. Thanks for paying attention to this feature.
Comment 9 Nelson Benitez 2008-05-27 09:17:50 UTC
Created attachment 111589 [details] [review]
Updated patch to nautilus svn trunk

Hi, I updated the patch to svn trunk and also ported it to gio, now it's working fine again.
Comment 10 Christian Neumair 2008-05-29 18:31:55 UTC
Thanks for your efforts.

Alex already made some comments back in 2006 on the patch, and some of them were still valid:

http://mail.gnome.org/archives/nautilus-list/2006-November/msg00044.html
http://mail.gnome.org/archives/nautilus-list/2006-December/msg00015.html

I committed a modified version of the patch:
http://svn.gnome.org/viewvc/nautilus?view=revision&revision=14208

Closing as fixed.