JDownloader Community - Appwork GmbH
 

 
 
Thread Tools Display Modes
  #1  
Old 01.06.2009, 04:45
ugkbunb
Guest
 
Posts: n/a
Default Open Directory Linux support for other file managers then Nautilus (Such as Thunar)

Open Directory only works in Linux if Nautilus is installed.... *Perhaps it works as well with Dolphin/Konqueror I have not tested it. I run xfce and installing nautilus while feasible is rather annoying. If this is a simple addition I would love to see support for other file managers most namely... Thunar, pcmanfm, and/or Rox.

great great program. many thanks in advance.

Last edited by Jiaz; 05.09.2009 at 21:09.
  #2  
Old 01.06.2009, 11:26
Jiaz's Avatar
Jiaz Jiaz is offline
JD Manager
 
Join Date: Mar 2009
Location: Germany
Posts: 79,544
Default

i will think about adding costuom filebrowser
__________________
JD-Dev & Server-Admin
  #3  
Old 01.09.2009, 19:18
ugkbunb
Guest
 
Posts: n/a
Default

bump... I would really love to see this feature implemented.

I know for thunar is it as simple as "thunar /directory/path/here"

I might take a stab at implementing it when I get home... Rather clueless when it comes to java but I might be able to manage a hackish patch for this simple feature request.
  #4  
Old 03.09.2009, 03:53
ugkbunb
Guest
 
Posts: n/a
Default

here is what I came up with... again I am very unexperienced with editing source files... but I believe this is correct:
Code:
--- GetExplorer.java	2009-09-02 15:49:47.676048503 -0500
+++ GetExplorerdiff.java	2009-09-02 20:51:29.101032018 -0500
@@ -35,7 +35,7 @@
         } else if (OSDetector.isMac()) {
             return new Object[] { "Open", "/usr/bin/open", new String[] { "%%path%%" } };
         } else {
-            Object[][] programms = new Object[][] { { "dolphin", new String[] { "%%path%%" } }, { "konqueror", new String[] { "%%path%%" } }, { "nautilus", new String[] { "--browser", "--no-desktop", "%%path%%" } } };
+            Object[][] programms = new Object[][] { { "dolphin", new String[] { "%%path%%" } }, { "konqueror", new String[] { "%%path%%" } }, { "thunar", new String[] { "%%path%%" } }, { "rox", new String[] { "%%path%%" } }, { "pcmanfm", new String[] { "%%path%%" } }, { "nautilus", new String[] { "--browser", "--no-desktop", "%%path%%" } } };
             try {
                 String[] charset = System.getenv("PATH").split(":");
                 for (String element : charset) {
I am not sure how to compile it to test if it is correct... goin to do some research on how to compile java programs within the ArchLinux PKGBUILD system... if I find out that this worked for me I will post confirming my patch.

Last edited by ugkbunb; 03.09.2009 at 03:55.
  #5  
Old 05.09.2009, 19:55
ugkbunb
Guest
 
Posts: n/a
Default

bump... can a developers look at my patch and perhaps include it in the next jdownloader update?
  #6  
Old 05.09.2009, 21:08
Jiaz's Avatar
Jiaz Jiaz is offline
JD Manager
 
Join Date: Mar 2009
Location: Germany
Posts: 79,544
Default

i added your patch, i think i will add a dropdown soon to choose
__________________
JD-Dev & Server-Admin
 

Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump

All times are GMT +2. The time now is 19:13.
Provided By AppWork GmbH | Privacy | Imprint
Parts of the Design are used from Kirsch designed by Andrew & Austin
Powered by vBulletin® Version 3.8.10 Beta 1
Copyright ©2000 - 2024, Jelsoft Enterprises Ltd.