- User Since
- Mar 20 2016, 4:24 PM (117 w, 3 d)
Thank you all! For my part, using Kubuntu 18.04: my performed tests went alright; some files (inside an iso file) that could not be seen, now can be seen correctly. Other people can do their checks.
Sun, Jun 17
Note: The tag "Krusader" was added to the present differential revision so that it's seen now at least in https://phabricator.kde.org/project/view/79/ and https://phabricator.kde.org/tag/krusader/ , and consequently more people may review this proposal.
Nice! There's more information in https://phabricator.kde.org/D13572
Thu, Jun 14
Was it changed for a reason?
I confirm what Nikita tried.
Wed, Jun 13
Also, using Krusader 2.4.0-beta3, when "files fed to list box" are deleted... they are physically deleted.
> Removal of items from virtual lists is not a very handful feature after all...
I think the current behavior is useful as is, i.e. keep the search active unless the dir is changed. Let's discuss more and
consider implementing an option to keep current behavior.
Tue, Jun 12
Mon, Jun 11
Many thanks, Nikita! For my part, my performed tests (using Kubuntu 18.04) went alright.
May 19 2018
Thanks! Davide's commit is available for viewing at https://commits.kde.org/krusader/eb60c734b940063f61064c5bcd25ae448b8aa610
May 18 2018
Thank you for your interest, Safa. Patches are welcome :-)
May 17 2018
For my part, my performed tests went also OK!
Just one minor suggestion: rename "Ugly message dialog on Compare by Content error" with "Fixed ugly message dialog on Compare by Content error" so others will know you haven't introduced anything ugly :)
For my part, my performed tests went alright!
Ctrl+Shift+S is available, where S is for selection. I'm also fine with the default Ctrl+G. Since users can easily remap, it's not a big deal.
May 11 2018
May 9 2018
I accept the proposal. Thanks, Yuri and Nikita!
May 8 2018
With the proposed change: things would be better, although Nikita's way of working would be an additional advantage. Thanks!
Check if anything else should be updated.
May 3 2018
The commit is available for viewing at https://commits.kde.org/websites/krusader-org/3d5726caeedc4fd544751202eb9e0f5930cfb218
Indeed. Maybe something like "Updated descriptions, author and thanks lists". This is what you did, isn't it?
May 2 2018
Thank you, Alex! For my part, my performed tests using Kubuntu 18.04 went alright, and I find it easier now to filter, to search, and to select. Great!
Some changes (proposed by Nikita Melnichenko) were made:
May 1 2018
A reference to KDE Plasma 4 was changed.
Some specifics were removed, in order to improve e.g. the maintainability.
Apr 30 2018
Two texts were changed to save time to some readers.
Two pages were created, and information was moved to those pages.
Apr 29 2018
Apr 27 2018
Still giving a +1 for moving this into KIO so Gwenview and Krusader can use it too if they'd like.
Thanks for the improvements, Nikita!
Apr 26 2018
The commit message was updated accordingly.
Some related changes were made, including one in a docbook file.
(independent of the patch) IMO, what a bad name scheme is picked for the actions. Names shouldn't contain default values. A better naming could be` FN_Rename` if one would like to denote these belong to the functional keys bar.
Apr 25 2018
Thanks for the work you put on it!
Apr 22 2018
Toni, please accept this revision if you think it's good to merge with master.
Pinging @asensi on whether this would be a good addition to the Krusader project so then we can consider moving it to KIO).
Apr 21 2018
To reproduce this issue: set a dark colours theme and oxygen icons, close all tabs in one panel and observe disabled icon in Window menu's Close Current Tab action. Try this with master
branch and with fix-missing-icons branch.
Apr 20 2018
More screenshots, using included icons in Kubuntu 17.10:
Using Ubuntu 18.04 daily build: I could reproduce the problem, and it was solved by the patch made by Nikita. Thanks, Nikita!
Apr 19 2018
Hello! I can't reproduce the "fixed invisible column" issue, so I attach some screenshots and "krusaderrc" files, if it may help:
With the latest improvements made by Nikita, In an Ubuntu 18.04 daily build virtual machine, and in a Kubuntu 17.10 virtual machine, the icons were shown as it was expected:
Apr 15 2018
Regarding the greyed out icons in Krusader: I believe this is because "breeze" is set
as system theme (not "breeze-dark") and fallback is also set as "breeze", so
there is no "breeze-dark" to fallback to.
Apr 14 2018
With the proposed changes:
Note: That "icon-missing.svgz" file can be downloaded from https://phabricator.kde.org/file/download/ieoxfxmcslybdhrcwkhf/PHID-FILE-zues6xeejyrafgjglyel/icon-missing.svgz
Thanks, Nikita! A lot of work! I also wanted to say that, after applying the diff file, using Ubuntu 18.04 daily build, I saw that error message when building Krusader (because the "icon-missing.svgz" file was not found):
[ 84%] Built target krusader_autogen make: *** No hay ninguna regla para construir el objetivo '/home/user/krusader/krusader/icons/icon-missing.svgz', necesario para 'krusader/qrc_resources.cpp'. Alto. CMakeFiles/Makefile2:222: recipe for target 'krusader/CMakeFiles/krusader.dir/all' failed make: *** [krusader/CMakeFiles/krusader.dir/all] Error 2 Makefile:140: recipe for target 'all' failed make: *** [all] Error 2
Apr 9 2018
I'm fine with "emblem-unreadable" although my preference would rather be "image-missing" :).
Apr 7 2018
Apr 6 2018
When using Kubuntu 17.10:
Apr 5 2018
Apr 4 2018
It worked as it is expected, OK!
Hi! I'm using the "bookmark-quicksearch" branch, and I have applied the present diff, which is:
diff --git a/krusader/BookMan/krbookmarkhandler.cpp b/krusader/BookMan/krbookmarkhandler.cpp --- a/krusader/BookMan/krbookmarkhandler.cpp +++ b/krusader/BookMan/krbookmarkhandler.cpp @@ -592,12 +592,12 @@ return true; }
Thanks for improving consistency!
I propose we add CHANGED: ... to commit message, so it goes to the ChangeLog.
Toni, thanks a lot for testing this! Quick question: is Breeze installed in both of these cases or not?
Apr 3 2018
In the same Ubuntu 18.04 daily build virtual machine, after applying the present differential:
Thank you all!
I also added a "tag" named "Krusader" and so:
My upgrade looks like this:
Qt: 5.7.1 -> 5.9.4
KF: 5.40 -> 5.43
kde-apps: 17.08.3 -> 17.12.3
Mar 26 2018
BTW, it's not just my compiler settings, Toni also confirmed this earlier [...]
Mar 11 2018
Thank you, Martin! For my part, my performed tests (using Kubuntu 17.10) went alright:
Thank you, Martin! For my part, my performed tests (using Kubuntu 17.10) went alright, including those tests that were already mentioned:
- go up in standard local FS
- go up outside an archive (with krarc protocol)
- go up outside an archive (with tar protocol)
Mar 10 2018
Thank you, Martin! (and Nikita :-)! For my part, my performed tests (using Kubuntu 17.10) went alright, including those already mentioned tests:
- pin a tab, browse in it, activate another tab in the same panel, reactivate pinned tab -> it should return to the pinned address
- pin a tab, restart Krusader -> pinning should still be active, same for saved profiles
- pin a tab, change directory in it, duplicate it -> check that original tab stays pinned and its title is not changed, check that duplicated tab is not pinned and its path is correct (meaning - not the pinned one)
Other people can do their checks :-)
Thank you, Martin! (and Nikita :-)! For my part, my performed tests (using Kubuntu 17.04) went alright. Other people can do their checks.
Feb 28 2018
Just to confirm, do you guys see the other 3 warnings with gcc < 7?
Feb 27 2018
It also works using gcc 5.4.0 under Ubuntu 16.04 LTS (its latest Long Term Service version). Thanks!
Feb 24 2018
I've changed delimiter to space and used KShell::splitArgs() method to allow using
- quoted folder names
- folder names with escaped spaces