ROX-Filer 2.11 patched

Filemanagers, partitioning tools, etc.
Message
Author
woodenshoe-wi
Posts: 109
Joined: Sat 29 Jul 2017, 03:16
Location: Wisconsin

#16 Post by woodenshoe-wi »

New version again, I tried to make a sfs but I couldn't get it to load over the main sfs so I repacked it as a pet with just the AppDir and desktop file. This one will not overwrite any settings, but should not be used for woof builds.
Edit: Newer versions posted later.

I was able to fix the "copy/paste" functionality. The problem was that the clipboard was "set" during initialization instead of when actually doing a "copy" or "cut", so it would stop working as soon as another program used the clipboard. Once I moved the "set" function to the proper place the strange behavior stopped.

For debugging I wrote some scripts that use xclip to copy and paste from the command line, I don't know if anyone else might find them useful?

g-copy

Code: Select all

#!/bin/bash

case $@ in
	-h|-*help|-*usage)
		echo "Usage: $(basename $0) <files to copy>"
		exit 0
		;;
	"")
		echo "Usage: $(basename $0) <files to copy>"
		exit 1
		;;
esac

clipboard="copy
"

for one_arg in "$@"
do
	#echo "X${one_arg}X"

	filename=$(realpath "${one_arg}")
	#echo "${filename}"

	clipboard="${clipboard}file://${filename}
"
done

echo "${clipboard}" | xclip -i -selection clipboard -t x-special/gnome-copied-files
g-cut

Code: Select all

#!/bin/bash

case $@ in
	-h|-*help|-*usage)
		echo "Usage: $(basename $0) <files to cut>"
		exit 0
		;;
	"")
		echo "Usage: $(basename $0) <files to cut>"
		exit 1
		;;
esac

clipboard="cut
"

for one_arg in "$@"
do
	#echo "X${one_arg}X"

	filename=$(realpath "${one_arg}")
	#echo "${filename}"

	clipboard="${clipboard}file://${filename}
"
done

echo "${clipboard}" | xclip -i -selection clipboard -t x-special/gnome-copied-files
and g-paste

Code: Select all

#!/bin/bash

carriage_return=$(echo -ne '\r')

filelist=$(xclip -o -selection clipboard -t x-special/gnome-copied-files)

paste_mode=$(echo "${filelist}" | head -n 1)
#echo "X${paste_mode}X"

while read line
do
	line=${line//${carriage_return}/}
	#echo "X${line}X"

	if [ "${line}" = "${paste_mode}" ]; then
		continue
	fi

	filename=${line#file:\/\/}
	#echo "${filename}"

	if [ -e "${filename}" ]; then
		if [ "${paste_mode}" = "cut" ]; then
			mv -f "${filename}" .
		else
			cp -af --no-preserve=context,links "${filename}" .
		fi
	else
		echo "Error parsing line: ${line}"
	fi

done <<< "${filelist}"
Pets that are suitable for woof builds are also available,
Edit: Newer versions posted later.
But as before, be warned that they will reset your desktop background, overwrite your pin-board and possibly other unwanted changes... They are meant for woof builds. Also @peebee please DON'T upload to ibiblio unless or until you are confident that they work properly. I am posting them for more thorough testing.

Edit: Added 64 bit standalone version.
Edit2: Removed old links, newer versions posted later.
Last edited by woodenshoe-wi on Fri 18 May 2018, 21:01, edited 2 times in total.

Puppyt
Posts: 907
Joined: Fri 09 May 2008, 23:37
Location: Moorooka, Queensland
Contact:

#17 Post by Puppyt »

Thanks so much woodenshoe-wi - rox column management active in TahrPup64 6.0.6. Was rehashing an old grumble on a recent thread, and your pet has salved a sore point in Rox usability. Cheers!
Re: How do I modify ROX detail columns?

UPDATE: Sorry, I have to qualify my test as a partial success only. I haven't tested copy'n'paste which I see was your main concern with your patch. However, I got a bit giddy with the detail view column change functionality, which in my TahrPup64 version had been locked into a useless Name/Permissions/Owner/Group/Type/Size/Last modified... format. Who needs Permissions/Owner/Group in Puppy??? Wish there was some way to eliminate those columns entirely.
Anyway, while your patch is great for shifting the ordered sequence of columns in any Rox window - the changes aren't persistent between Rox windows, and are not persistent on reboots. Any hope for tweaking your patch just a bit further for permanence?
Search engines for Puppy
[url]http://puppylinux.us/psearch.html[/url]; [url=https://cse.google.com/cse?cx=015995643981050743583%3Aabvzbibgzxo&q=#gsc.tab=0]Google Custom Search[/url]; [url]http://wellminded.net63.net/[/url] others TBA...

woodenshoe-wi
Posts: 109
Joined: Sat 29 Jul 2017, 03:16
Location: Wisconsin

#18 Post by woodenshoe-wi »

Well, first I want to make it clear that I didn't write the movable columns patch. I cherry picked a commit by JakeSFR who gave the credit to forum users Otter & Rocket.
https://github.com/woodenshoe-wi/rox-fi ... 20f7735bc4

The copy and paste functionality was actually something I saw in a GitHub fork of rox-filer by dtomas https://github.com/dtomas/rox-filer
It did not look like it had been merged into the jun7 fork, but it would solve a complaint that rox-filer lacks proper copy and paste support. When I added "cut to clipboard" support I accidentally added a segfault, which I fixed when I found it, and I also made changes to the original implementation of "copy and paste" to fix problems I found with it.

My original motivation for making these changes was to get the "overwrite if newer" check box working properly because there were two camps of ROX users, those including myself who were happy with the old behavior of being prompted about every overwrite, and those that found the prompts irritating and preferred to use a newer version of ROX that didn't prompt when overwriting regardless of whether the overwriting file was newer or older than the file that was being overwritten.

When I was working on this I also noticed that the function that handled moving files and directories had diverged from the function that copies files and directories so I rewrote the "move" function to more closely match the "copy" function and now it should be possible to "merge" directories when moving them just like when copying them.

Because of all these changes and the potential for bugs and segfaults I would like people to test this version before it would be made a default version in woof builds, but I am not sure how to go about getting it tested. There are no "testing" version package lists, once it would be uploaded to ibiblio it could end up being the default version which would be bad if it had any serious bugs.

woodenshoe-wi
Posts: 109
Joined: Sat 29 Jul 2017, 03:16
Location: Wisconsin

#19 Post by woodenshoe-wi »

New version again. These are the standalone versions that will not overwrite any settings, but should not be used for woof builds.

Edit: These were buggy, newer versions posted later.

This version makes NO changes to rox column management, but instead adds the feature that when a directory is open in a filer window and the directory no longer exists because it is unmounted or deleted the filer window will be automatically closed.

Pets that are suitable for woof builds are also available,
Edit: These were buggy, newer versions posted later.
But as before, be warned that they will reset your desktop background, overwrite your pin-board and possibly other unwanted changes... They are meant for woof builds. Also @peebee please DON'T upload to ibiblio unless or until you are confident that they work properly. I am posting them for more thorough testing.

It should also be possible to use my gitlab repo for test woof builds by All 3 steps are needed.

Starting with version 6, the name that has to be added to DISTRO_PKGS_SPECS is no longer "rox-filer-woodenshoe", just the standard "rox-filer".
Last edited by woodenshoe-wi on Mon 02 Jul 2018, 05:26, edited 2 times in total.

woodenshoe-wi
Posts: 109
Joined: Sat 29 Jul 2017, 03:16
Location: Wisconsin

#20 Post by woodenshoe-wi »

Version 2 was buggy, this new version can also configure which columns are visible in "List View".

Edit: Newer versions posted later.

Pets that are suitable for woof builds are also available,
Edit: Newer versions posted later.
But as before, be warned that they will reset your desktop background, overwrite your pin-board and possibly other unwanted changes... They are meant for woof builds. Also @peebee please DON'T upload to ibiblio unless or until you are confident that they work properly. I am posting them for more thorough testing.
Last edited by woodenshoe-wi on Thu 24 May 2018, 05:50, edited 1 time in total.

User avatar
davids45
Posts: 1326
Joined: Sun 26 Nov 2006, 23:33
Location: Chatswood, NSW

Pinboard icon text

#21 Post by davids45 »

G'day woodenshoe-wi,

In response to your post on the Rox poll thread :) , I had just tried your latest .pet for your Rox in a BionicPup1805-2+Sc64bit-kernel Frugal.

The new Rox column adjustability is better - I deleted the never-been-used 'Permissions', 'Owner' and 'Group' columns (screenshot).

This Pup was already fully apped with my apps-links sfs and the pinboard icon labels edited by me, these apps having been dragged as .desktop files from /usr/share/applications/ so I remove '.desktop' and sometimes shorten what's left to fit under the icon.

1) with the new Rox, no .desktop to worry about but some application icon labels became too long and overlapped text of adjacent icons (screenshot)

2) when I tried to edit the too-long labels, I couldn't with your Rox. This may be something due to these being links to my data partition, not a 'normal' file on my frugals partition?

Can I edit something to shorten the over-long labels before they're being loaded onto the pinboard?

I found nothing else an issue with some quick tests with your Rox but have reverted to my previous savefile with its older Rox version and icon-text-editable pinboard (screenshot).

Happy to try anything else when I can.

Thanks for your work with Rox,

David S.
Attachments
newRox-details-columns.jpg
unwanted/unused columns gone from new Rox details window
(75.24 KiB) Downloaded 1030 times
newRox-pinboard.jpg
application icon labels (some) too long and overlapping, which I could not edit away with the new Rox.
(117.72 KiB) Downloaded 1055 times
normal-pinboard-icon-labels.jpg
pre-new-Rox pinboard with application icon labels after normal removal of '.desktop' - works with 'old' Rox
(113.33 KiB) Downloaded 1046 times

woodenshoe-wi
Posts: 109
Joined: Sat 29 Jul 2017, 03:16
Location: Wisconsin

#22 Post by woodenshoe-wi »

Thank you for testing davids45, I don't normally link .desktop files to my pinboard so I wouldn't have found the bug by myself.

Looking into the problem it seems that when the feature to use the name from .desktop files was added it would always override any user edits.

Here are new pets with a fix applied:
Edit: Newer versions posted later.

Pets that are suitable for woof builds are also available,
32 bit version https://gitlab.com/woodenshoe-wi/petbui ... s-common32
64 bit version https://gitlab.com/woodenshoe-wi/petbui ... s-common64
But as before, be warned that they will reset your desktop background, overwrite your pin-board and possibly other unwanted changes... They are meant for woof builds.
Last edited by woodenshoe-wi on Sun 17 Jun 2018, 23:35, edited 1 time in total.

User avatar
recobayu
Posts: 387
Joined: Wed 15 Sep 2010, 22:48
Location: indonesia

#23 Post by recobayu »

Thank you, woodenshoe-wi.
I download your rox standalone and common. I install it on bionicpup. I got error on common's rox. I can not click on toolbar and i can not right click. But it does not happen with rox standalone in my bionicpup.
I am very happy because now i can make new directory by clicking on '+' toolbar. Also i can hide the unused column in list view.

I also thank you. Because I can copy-paste multiple file with yor rox-filer.

I still have problem when i will select multiple file by mouse (block by click and drag) in list view mode. I must choose the right coordinate (usually a line between two file) so i can block it. Could it be more easier as icon view does?


But I say thank you again for your hard work.
:)

woodenshoe-wi
Posts: 109
Joined: Sat 29 Jul 2017, 03:16
Location: Wisconsin

#24 Post by woodenshoe-wi »

recobayu wrote:I got error on common's rox. I can not click on toolbar and i can not right click. But it does not happen with rox standalone in my bionicpup.
I can reproduce the problem you describe, restarting the X server fixes it for me. The strange thing is that I make the "standalone" version by deleting the files in /etc, /root and the /usr/local/bin/rox script from the "common" pet.

Maybe something is wrong with one of the config files in the petbuild template. The "common" pets are meant for woof builds, and the standalone pets are meant for installing in a running pup. Other than the configuration files they are identical.

I have an idea to make selecting files in list view easier by only allowing the icon and name columns to drag items, but I have to solve the "Attempt to paste empty selection!" warning first. So I won't be posting anything to test until I get this figured out.

In the meantime you could try holding down the CTRL key and clicking on each item you want to select. Time consuming, but maybe more reliable than clicking in just the right place?

User avatar
Smithy
Posts: 1151
Joined: Mon 12 Dec 2011, 11:17

#25 Post by Smithy »

Hello woodenshoe, is there somewhere in your code to:

1. Remove the Copy dialogue completely (the shift+Ctrl+C one)
2. Rename Copy to Clipboard just to Copy.
3. Rename Cut to Clipboard just to Cut.
4 Make the delete dialogue operate with the Delete button on keyboard
as opposed to the laborious Shift+Ctrl+X
5. Rename Paste to Clipboard to just Paste.

6. And finally have them grouped in the same area as the rename and delete section.

Been testing those items and they are rock solid, but just would like to remove the duplication (Copy) and move them right next to each other to slim down the right click a bit.
Attachments
woodenshoe rox1.png
(106.22 KiB) Downloaded 840 times

step
Posts: 1349
Joined: Fri 04 May 2012, 11:20

#26 Post by step »

Hi Smithy, two things you asked about, renaming a menu item and changing its hot key, can be done also without changing the source code. Of course, changing the source code is another way.
I tested these methods on Fatdog64 ROX-Filer, I don't know if the version in this thread behaves the same way, but my guess is that it does.

To change a hotkey select (don't click) the menu option that you want to change and press the key you want for it. For instance, the picture you posted is exactly at the right spot for changing Ctrl+Shift+C to F5. Just press Del once (to free Ctrl+Shift+C) and press F5 (to assign F5). You should see the menu entry change to "Copy... (F5)".

Changing the text of menu entries is more complicated, but achievable. The gist of it is to add a custom translation file inside the ROX-Filer application folder, for your case, exactly adding file /usr/local/apps/ROX-Filer/Messages/en/LC_MESSAGES/ROX-Filer.mo. How to create ROX-Filer.mo. That's the more complicated part, if you have no previous experience with translating Puppy Linux. PM me if you're interested. No offense taken if you aren't.
Last edited by step on Tue 12 Jun 2018, 21:07, edited 1 time in total.
[url=http://murga-linux.com/puppy/viewtopic.php?t=117546]Fatdog64-810[/url]|[url=http://goo.gl/hqZtiB]+Packages[/url]|[url=http://goo.gl/6dbEzT]Kodi[/url]|[url=http://goo.gl/JQC4Vz]gtkmenuplus[/url]

woodenshoe-wi
Posts: 109
Joined: Sat 29 Jul 2017, 03:16
Location: Wisconsin

#27 Post by woodenshoe-wi »

Smithy wrote: 1. Remove the Copy dialogue completely (the shift+Ctrl+C one)
I use the Copy dialogue to make backup copies of files that I am going to edit, so I am not willing to remove it but I could rename it to Duplicate to free up the Copy name. The Rename dialogue is based on the Move dialogue so Duplicate would be the equivalent.
Smithy wrote: 4 Make the delete dialogue operate with the Delete button on keyboard
as opposed to the laborious Shift+Ctrl+X
I would love to do that but I can't figure out how to get GTK to assign the Delete button as a shortcut from the source code. :(
Smithy wrote: 6. And finally have them grouped in the same area as the rename and delete section.
As long as you are OK with Paste staying where it is that sounds like a plan. I will work on it when I have some more time, so it won't be right away.

step
Posts: 1349
Joined: Fri 04 May 2012, 11:20

#28 Post by step »

woodenshoe-wi wrote:
Smithy wrote: 4 Make the delete dialogue operate with the Delete button on keyboard
as opposed to the laborious Shift+Ctrl+X
I would love to do that but I can't figure out how to get GTK to assign the Delete button as a shortcut from the source code. :(
In my copy of the source code, the shortcut key is assigned in menu.c

Code: Select all

{">" N_("Copy..."),		"<Ctrl>C", file_op, FILE_COPY_ITEM, "<StockItem>", GTK_STOCK_COPY},
If changing it there doesn't work then it could be set in a configuration file, either in the user's or XDG's configuration directories. In Fatdog64 those are ~/.config.mine/rox.sourceforge.net/ROX-Filer/menus2 and /etc/xdg/rox.sourceforge.net/ROX-Filer/menus2 respectively. My user's menu entry for copy is

Code: Select all

(gtk_accel_path "<filer>/File/Copy..." "F5")
which assigns F5 and overrides menu.c
[url=http://murga-linux.com/puppy/viewtopic.php?t=117546]Fatdog64-810[/url]|[url=http://goo.gl/hqZtiB]+Packages[/url]|[url=http://goo.gl/6dbEzT]Kodi[/url]|[url=http://goo.gl/JQC4Vz]gtkmenuplus[/url]

User avatar
Smithy
Posts: 1151
Joined: Mon 12 Dec 2011, 11:17

#29 Post by Smithy »

Hi step, woodenshoe, first of all, reading through the information in both of your posts, I was just wondering if rox filer had a script or suchlike, where I could chop for example the copy line out at line number xx and shift the text a bit. Clearly not, looking inside.

I only compile the odd driver in a blue moon and those lc translations and .mo files down in the bowels of puppy I just have a quick look to make sure that apps have a bit of internationalisation if I'm making one.

I only wanted to do that for a user perspective copy, (even though this version is in final music2go sfs that I put together).

I realise that probably loads of puppy devs and users rely on the Shift+Ctrl+C and probably other key commands, so I didn't mean for you to change your official version :)

If the delete key eventually becomes delete that will be great, the rest of my bullet points are really just cosmetic and a lot of the time I am ctrl c, ctrl x and ctrl v and of course...delete.

Actually duplicate could be a good idea woodenshoe, sort of makes sense.

I would say this rox filer makes things about 50% faster than the old drag and dropper that we have had to live with for years.. and got rid of the glipper thing on the taskbar.

woodenshoe-wi
Posts: 109
Joined: Sat 29 Jul 2017, 03:16
Location: Wisconsin

#30 Post by woodenshoe-wi »

@step I knew where the key binding was set but I couldn't figure out how to tell GTK to use the 'Delete' key. I had tried "<Del>" and "<Delete>" but it wouldn't recognize it. Somehow your post inspired me to try "Delete" and success!. I have the 'Delete' key assigned to the 'Delete' dialogue!
Smithy wrote:I realise that probably loads of puppy devs and users rely on the Shift+Ctrl+C and probably other key commands, so I didn't mean for you to change your official version :)
No... actually the original key binding for the old 'Copy' was Ctrl+C but I changed it to Shift+Ctrl+C because I wanted to use Ctrl+C for 'Copy to clipboard'. I think I will make Shift+Ctrl+C the key binding for 'Duplicate'.

Maybe I will have time to make a release this weekend.

User avatar
Puppus Dogfellow
Posts: 1667
Joined: Tue 08 Jan 2013, 01:39
Location: nyc

#31 Post by Puppus Dogfellow »

thanks for this, woodenshoe-wi.

working well in an old xenial32 spin (based on 704) and a few tahr64 spins (based on 605).

had backups of the pinboard(s) so having mine reset to the alien default wasn't a problem; being able to remove and move columns is great; the installation preserved all my other rox mods (custom icons and the ability to set right click shortcuts were added to the original iso and not lost when the new items were added to the tool bar and menus); the clipboard is great....


mirrored your efforts here: rox filer 4 by woodenshoe (32/64)

thanks again!

-pd

woodenshoe-wi
Posts: 109
Joined: Sat 29 Jul 2017, 03:16
Location: Wisconsin

#32 Post by woodenshoe-wi »

Here are new pets with the 'Delete' key binding change and menu rearranging. I decided to go with "Ctrl+D" for 'Duplicate' because that makes the menu a little narrower and is easier to type.
32 bit version https://www.dropbox.com/s/7xpy3m3nlhmqn ... 5.pet?dl=1
64 bit version https://www.dropbox.com/s/yghhwqcwejqwi ... 5.pet?dl=1

I also made some changes to the copy and move functions to bring them closer in line with the jun7 branch. If you don't have "Quite" mode default for the 'Copy' and 'Move' dialogues it will still prompt you if overwriting an older file with a newer one even if the 'Newer' check box is ticked. Jun7 figures it is a bug if you want to be prompted for every action (you are not in "Quite" mode) and older files get overwritten by newer ones without being prompted even though you are prompted for every move or copy that doesn't result in an overwrite. Makes sense to me, so this change is intentional.

Pets that are suitable for woof builds are available upon request, but they are not meant for installing into a running system so I am not posting links here.

User avatar
Smithy
Posts: 1151
Joined: Mon 12 Dec 2011, 11:17

#33 Post by Smithy »

Seems to be working good Woodenshoe, Not noticed any bugs in cut, copy paste.
What are roxapps btw? Can one make videoplayers and audio players for rox? I think pizzagood's ram thing was a roxapp?
Too lazy to search through loads of..text.

woodenshoe-wi
Posts: 109
Joined: Sat 29 Jul 2017, 03:16
Location: Wisconsin

#34 Post by woodenshoe-wi »

Smithy wrote:What are roxapps btw? Can one make videoplayers and audio players for rox? I think pizzagood's ram thing was a roxapp?
Too lazy to search through loads of..text.
I'm afraid I can't help you there, I don't know anything about roxapps myself.

Unless it is the same thing as an AppDir which just means that it is a directory with an AppRun script inside and unless you hold down the "Shift" key when you click on it ROX will run the AppRun script instead of opening the directory.

Directories can also have a .DirIcon file inside which will be shown instead of the standard directory icon for that particular directory, and this version of ROX has the patch applied to allow files to have "comment" files.
Attachments
comment_DirIcon.png
(37.7 KiB) Downloaded 767 times

woodenshoe-wi
Posts: 109
Joined: Sat 29 Jul 2017, 03:16
Location: Wisconsin

#35 Post by woodenshoe-wi »

New version for testing,
32 bit version https://www.dropbox.com/s/fvhsutzp1xu8f ... e.pet?dl=1
64 bit version https://www.dropbox.com/s/eaoh8kw74wfgk ... e.pet?dl=1

These have some significant changes in the way that filer windows are closed if the directory they are showing is missing/deleted. It is now a setting in 'Options...' if you want to close the window automatically.

Post Reply