Feed Advanced Search

Aug 21 2019

asensi added inline comments to D23309: Change the document about helping Krusader, add other ways.
Aug 21 2019, 9:29 PM · Krusader, Documentation
pino added a comment to D23309: Change the document about helping Krusader, add other ways.

Note: There are some reasons why Krusader users have documentation available in their computers, maybe Yuri Chornoivan can give more information about it.

Aug 21 2019, 7:17 AM · Krusader, Documentation
yurchor accepted D23309: Change the document about helping Krusader, add other ways.

Personally, I think that there will be no harm if we have as many ways to engage new developers as we can.

Aug 21 2019, 6:39 AM · Krusader, Documentation
asensi added a comment to D23309: Change the document about helping Krusader, add other ways.
In D23309#515717, @pino wrote:

TBH I'd just add the instructions to https://krusader.org/get-involved/, and not to the user documentation.

Aug 21 2019, 6:32 AM · Krusader, Documentation
yurchor accepted D23276: Add more compressed file types to the list of file types that krarc supports.
Aug 21 2019, 5:19 AM · Krusader
pino added a comment to D23309: Change the document about helping Krusader, add other ways.

TBH I'd just add the instructions to https://krusader.org/get-involved/, and not to the user documentation.
Just like we do not add compilation/build instructions to documentations anymore, IMHO it makes sense to not add contribution instructions either: they are generally not related to "using the application", and thus the old documentation shipped with an old version would show outdated links. Some of the changes in this patch perfectly show this: imagine that users installing krusader in stable distros have outdated links to freshmeat and linux-apps in the documentation they read.

Aug 21 2019, 5:03 AM · Krusader, Documentation

Aug 20 2019

asensi updated the diff for D23276: Add more compressed file types to the list of file types that krarc supports.

Some spaces were added, as Yuri suggested.

Aug 20 2019, 10:01 PM · Krusader
asensi added inline comments to D23276: Add more compressed file types to the list of file types that krarc supports.
Aug 20 2019, 9:59 PM · Krusader
asensi requested review of D23309: Change the document about helping Krusader, add other ways.
Aug 20 2019, 9:53 PM · Krusader, Documentation
yurchor added inline comments to D23276: Add more compressed file types to the list of file types that krarc supports.
Aug 20 2019, 5:34 AM · Krusader

Aug 19 2019

asensi added a comment to D23001: For the "kde4" branch: 08 - Update the synchronizer.docbook file because of the latest changes REVIEW: 124335.

How do you think a person is going to search whether kde4 version of krusader is getting updates? Will she likely go to a search engine and type something like "krusader kde4" where the mentioned repo will be in top positions due to good naming?

Aug 19 2019, 9:17 PM · Krusader, Documentation
asensi updated the summary of D23276: Add more compressed file types to the list of file types that krarc supports.
Aug 19 2019, 9:07 PM · Krusader
asensi requested review of D23276: Add more compressed file types to the list of file types that krarc supports.
Aug 19 2019, 9:05 PM · Krusader
gengisdave accepted D23236: Updates related to v2.7.2 release.
Aug 19 2019, 7:44 PM · Krusader
gengisdave accepted D23235: Released v2.7.2.
Aug 19 2019, 7:43 PM · Krusader
gengisdave accepted D23197: Updated ChangeLog and NEWS with v2.7.2 changes.
Aug 19 2019, 7:43 PM · Krusader
nmel added a comment to D23001: For the "kde4" branch: 08 - Update the synchronizer.docbook file because of the latest changes REVIEW: 124335.

For example, you can fork krusader on github or other platform [...]. What do you think about this idea, Toni?

I don't think people would find it, Nikita.

Aug 19 2019, 6:17 AM · Krusader, Documentation

Aug 18 2019

asensi accepted D23235: Released v2.7.2.

It looks OK for me, thanks for it, Nikita!

Aug 18 2019, 8:30 PM · Krusader
asensi accepted D23236: Updates related to v2.7.2 release.

It looks OK for me, thanks for it, Nikita!

Aug 18 2019, 8:30 PM · Krusader
asensi added a comment to D23001: For the "kde4" branch: 08 - Update the synchronizer.docbook file because of the latest changes REVIEW: 124335.

For example, you can fork krusader on github or other platform [...]. What do you think about this idea, Toni?

Aug 18 2019, 8:21 PM · Krusader, Documentation
yurchor accepted D23197: Updated ChangeLog and NEWS with v2.7.2 changes.

Accepted for my part. Thanks.

Aug 18 2019, 8:21 PM · Krusader
asensi accepted D23197: Updated ChangeLog and NEWS with v2.7.2 changes.

For my part, I accept the proposal, Thanks for it, Nikita!

Aug 18 2019, 7:57 PM · Krusader
asensi added a comment to D23197: Updated ChangeLog and NEWS with v2.7.2 changes.

Well, some months have passed since Davide's commit :-( , let's see when normal Krusader users can take profit.

Aug 18 2019, 7:50 PM · Krusader
asensi added a comment to D22957: Replace Q_WS_WIN with Q_OS_WIN in all the source code.
In D22957#513827, @nmel wrote:

This is a very serious change because now #ifndef Q_OS_WIN or #else block after #ifdef Q_OS_WIN will include the code that previously was ignored. We must look at all those code paths and test features it affects. BTW, does it mean that some of those code paths do not work correctly in the current state?

Aug 18 2019, 7:41 PM · Krusader
asensi updated the summary of D22957: Replace Q_WS_WIN with Q_OS_WIN in all the source code.
Aug 18 2019, 7:33 PM · Krusader
asensi updated subscribers of D23243: Avoid that Krusader crashes when trying to view any archive using the ark part.
Aug 18 2019, 7:24 PM · Krusader
asensi requested review of D23243: Avoid that Krusader crashes when trying to view any archive using the ark part.
Aug 18 2019, 7:20 PM · Krusader
nmel added a comment to D22434: Replace obsolete QFontMetrics::width with QFontMetrics::horizontalAdvance.

When I see the usage of the method, like

const float fontWidth = (fm.QFONTMETRICS_WIDTH("WWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWW") - fm.QFONTMETRICS_WIDTH("W")) / 99.0;

or

headerView->resizeSection(KrViewProperties::Ext, QFontMetrics(_viewFont).QFONTMETRICS_WIDTH("tar.bz2  "));
headerView->resizeSection(KrViewProperties::KrPermissions, QFontMetrics(_viewFont).QFONTMETRICS_WIDTH("rwx  "));
headerView->resizeSection(KrViewProperties::Size, QFontMetrics(_viewFont).QFONTMETRICS_WIDTH("9") * 10);

which are all dirty hacks, I doubt we need to care too much about the differences mentioned. This code needs to be refactored properly but it's not a goal of this change. QFONTMETRICS_WIDTH will actually become a good marker of the places that need review if someone will find time and courage to make it right one day...

Aug 18 2019, 6:47 AM · Krusader
nmel added a comment to D22957: Replace Q_WS_WIN with Q_OS_WIN in all the source code.

This is a very serious change because now #ifndef Q_OS_WIN or #else block after #ifdef Q_OS_WIN will include the code that previously was ignored. We must look at all those code paths and test features it affects. BTW, does it mean that some of those code paths do not work correctly in the current state?

Aug 18 2019, 6:37 AM · Krusader
nmel added a comment to D23118: Changed shortcut for the embedded terminal emulator.

Good catch, Toni! No objections for the new shortcut. Please look into

  1. Making the commit headline shorter. Something like "Changed shortcut for the embedded terminal emulator". It's easier to read in the graph view. Please keep everything else, it's very good to have a detailed explanation in the commit message!
  2. Specifying CHANGED: tag, so this is not missed in the ChangeLog on the next release.
Aug 18 2019, 6:30 AM · Krusader
nmel added a comment to D23001: For the "kde4" branch: 08 - Update the synchronizer.docbook file because of the latest changes REVIEW: 124335.

I have no problem when someone is trying to improve a dead branch. If the person finds this useful and wants to share their changes to others — he or she is welcome to do so. It's an Open Source after all. In the same time, I understand what Luigi is saying that it might send a wrong signal to users. In addition, commits we push to the official repo are reviewed and tested, and should be only pushed when approved by at least another dev and no objections from others (unfortunately, it's not enforced due to a weak infrastructure). It means someone needs to test the changes you propose in 10 code reviews. Personally, I have no interest in kde4 commits anymore and even have no environment to test it. I doubt you'll find another dev who has.

Aug 18 2019, 6:21 AM · Krusader, Documentation
nmel added a project to D23236: Updates related to v2.7.2 release: Krusader.

Related review: D23197 (same changelog)

Aug 18 2019, 5:59 AM · Krusader
nmel added a project to D23235: Released v2.7.2: Krusader.
Aug 18 2019, 5:56 AM · Krusader

Aug 17 2019

nmel added a comment to D23197: Updated ChangeLog and NEWS with v2.7.2 changes.

Hi Toni,

Aug 17 2019, 5:42 AM · Krusader

Aug 16 2019

asensi added a comment to D23197: Updated ChangeLog and NEWS with v2.7.2 changes.

Thanks, Nikita! Because it's very handy, and it has been already tried a lot, it looks like a good idea to add the "Shortcuts to move tab" commit to the stable branch, and add to the ChangeLog file:

FIXED: [ 328937 ] Shortcuts for move tab
Aug 16 2019, 8:55 PM · Krusader
nmel edited the content of Release Howto.
Aug 16 2019, 5:59 AM · Krusader
nmel added a project to D23197: Updated ChangeLog and NEWS with v2.7.2 changes: Krusader.
Aug 16 2019, 5:38 AM · Krusader

Aug 12 2019

asensi retitled D23118: Changed shortcut for the embedded terminal emulator from Because Ctrl+Alt+T is now a global shortcut, use another one for the embedded terminal emulator to Because Ctrl+Alt+T is now a global shortcut, use another shortcut for the embedded terminal emulator.
Aug 12 2019, 9:53 PM · Krusader
asensi requested review of D23118: Changed shortcut for the embedded terminal emulator.
Aug 12 2019, 9:52 PM · Krusader
asensi closed D22883: Clarify the aim of two lines written recently by David Faure.
Aug 12 2019, 9:24 PM · Krusader
asensi added a comment to D22883: Clarify the aim of two lines written recently by David Faure.

Ok. Thanks, Alex and Davide!

Aug 12 2019, 9:24 PM · Krusader
gengisdave accepted D22883: Clarify the aim of two lines written recently by David Faure.
Aug 12 2019, 7:52 PM · Krusader
gengisdave updated subscribers of D22434: Replace obsolete QFontMetrics::width with QFontMetrics::horizontalAdvance.
Aug 12 2019, 7:50 PM · Krusader
gengisdave updated subscribers of D22434: Replace obsolete QFontMetrics::width with QFontMetrics::horizontalAdvance.
Aug 12 2019, 7:49 PM · Krusader
gengisdave updated subscribers of D22434: Replace obsolete QFontMetrics::width with QFontMetrics::horizontalAdvance.

An article about it has been published recently:

About deprecation of QFontMetrics::width()
https://kdepepo.wordpress.com/2019/08/05/about-deprecation-of-qfontmetricswidth/

by @cfeck (Christoph Feck, who has commited some changes to the repository of Krusader).

Aug 12 2019, 7:49 PM · Krusader

Aug 10 2019

asensi edited the content of Release Howto.
Aug 10 2019, 6:38 AM · Krusader

Aug 9 2019

asensi edited the content of Release Howto.
Aug 9 2019, 10:20 PM · Krusader
asensi edited the content of Release Howto.
Aug 9 2019, 10:12 PM · Krusader
nmel closed D22982: Updated release information in docs.
Aug 9 2019, 6:02 AM · Krusader, Documentation
asensi accepted D22982: Updated release information in docs.

Thanks, Nikita!

Aug 9 2019, 5:57 AM · Krusader, Documentation
nmel edited the content of Krusader.
Aug 9 2019, 5:55 AM · Krusader
asensi updated subscribers of D22434: Replace obsolete QFontMetrics::width with QFontMetrics::horizontalAdvance.
Aug 9 2019, 5:42 AM · Krusader
nmel added a comment to D22982: Updated release information in docs.

Updated AppStream files. Thanks Luigi and Toni!

Aug 9 2019, 5:41 AM · Krusader, Documentation
yurchor accepted D22982: Updated release information in docs.
Aug 9 2019, 5:41 AM · Krusader, Documentation
nmel updated the diff for D22982: Updated release information in docs.
  • Updated AppStream files with previous and upcoming release info
Aug 9 2019, 5:40 AM · Krusader, Documentation

Aug 8 2019

asensi updated subscribers of D22434: Replace obsolete QFontMetrics::width with QFontMetrics::horizontalAdvance.

An article about it has been published recently:

About deprecation of QFontMetrics::width()
https://kdepepo.wordpress.com/2019/08/05/about-deprecation-of-qfontmetricswidth/

by @cfeck (Christoph Feck, who has commited some changes to the repository of Krusader).

Aug 8 2019, 10:20 PM · Krusader
asensi added a comment to D22982: Updated release information in docs.

Please also add the versioning data to the appdata file (maybe also add the information for the previous version this time):

Aug 8 2019, 8:48 PM · Krusader, Documentation

Aug 7 2019

asensi added a comment to D23001: For the "kde4" branch: 08 - Update the synchronizer.docbook file because of the latest changes REVIEW: 124335.

Hi, Luigi! Well, I was referring to the virtual machines running prior operating systems (and physical machines, I remember that the LTS support of Ubuntu 14.04 ended at the end of April and I still saw some of those machines). Sometimes the reason is that some programs do not run in recent versions of operating systems, other causes may exist.
There's also the case of Lliurex 16, an operating system installed in thousands of computers, I had to work with it and there I could only build the "kde4" version of Krusader.

Aug 7 2019, 3:41 PM · Krusader, Documentation
ltoscano added a comment to D23001: For the "kde4" branch: 08 - Update the synchronizer.docbook file because of the latest changes REVIEW: 124335.

I'm not a Krusader developer, but from the global point of view, the kde4 branch is basically dead. It's not tracked for translations, it's not tracked by the sysadmin/repo-metadata repository.

Aug 7 2019, 3:17 PM · Krusader, Documentation
asensi requested review of D23003: For the "kde4" branch: 10 - Avoid closing the tab when the user {presses Esc inside the Krusader internal editor in order to try to exit searching}; thanks to Martin Kotelnik.
Aug 7 2019, 3:12 PM · Krusader
asensi requested review of D23002: For the "kde4" branch: 09 - Make Alt+. the default shortcut for toggling hidden files.
Aug 7 2019, 3:12 PM · Krusader, Documentation
asensi requested review of D23001: For the "kde4" branch: 08 - Update the synchronizer.docbook file because of the latest changes REVIEW: 124335.
Aug 7 2019, 3:12 PM · Krusader, Documentation
asensi requested review of D23000: For the "kde4" branch: 07 - In the synchronizer: until the bug 270150 is solved, stop using protocols like FISH, SFTP, SMB, etc. Check if the target or the source directory are empty.
Aug 7 2019, 3:11 PM · Krusader
asensi requested review of D22999: For the "kde4" branch: 06 - Reverse defaults for Synchronizer.
Aug 7 2019, 3:11 PM · Krusader
asensi requested review of D22998: For the "kde4" branch: 05 - Fix definition scope for Synchronizer in CMake.
Aug 7 2019, 3:11 PM · Krusader
asensi requested review of D22997: For the "kde4" branch: 04 - Update the main CMakeLists.txt to enable by default the building of synchronizer.
Aug 7 2019, 3:10 PM · Krusader
asensi requested review of D22996: For the "kde4" branch: 03 - Use the same font for file name and renamed file name; solved by Tobias BUG: 326649.
Aug 7 2019, 3:10 PM · Krusader
asensi requested review of D22995: For the "kde4" branch: 02 - Names of files are correctly seen when a zip file is opened, even if they are utf-8 encoded.
Aug 7 2019, 3:10 PM · Krusader
asensi requested review of D22994: For the "kde4" branch: 01 - Solve a bug that didn't allow Krusader to search files inside zip archives; thanks to Dawid Weiss BUG: 292867 REVIEW: 124343.
Aug 7 2019, 3:10 PM · Krusader
ltoscano added a comment to D22982: Updated release information in docs.

Please also add the versioning data to the appdata file (maybe also add the information for the previous version this time):
https://community.kde.org/ReleasingSoftware#Versioning_in_AppStream_files

Aug 7 2019, 7:33 AM · Krusader, Documentation
nmel added a comment to D22982: Updated release information in docs.

@yurchor, this was fast! Thanks!

Aug 7 2019, 6:06 AM · Krusader, Documentation
nmel added a project to D22982: Updated release information in docs: Krusader.

Please review by August 10, EOD. Thanks!

Aug 7 2019, 6:05 AM · Krusader, Documentation

Aug 5 2019

asensi requested review of D22957: Replace Q_WS_WIN with Q_OS_WIN in all the source code.
Aug 5 2019, 9:06 PM · Krusader
asensi accepted D22932: Remove 'isExiting' logic from closing sequence.

The new code works using Kubuntu 18.04 and 19.04. Krusader behaves as expected when it's asked to be closed:

  • when its internal editor is not used,
  • when its internal editor has a file that is not saved,
  • when its internal editor has a file that is not saved and the computer is asked to reboot.
Aug 5 2019, 5:23 AM · Krusader

Aug 4 2019

asensi added a comment to D22891: If an item that is going to be renamed is also going to be hidden by default, then warn the user.

the target audience of Krusader are more advanced users

Aug 4 2019, 10:28 PM · Krusader
asensi added a comment to D22666: Achieve that Krusader searches correctly if the "Follow links" checkbox is disabled.

If any developer is reading this, he'll probably know that those developments end up being a very complex subject. Thank you for all the source code, Alex!

Aug 4 2019, 10:14 PM · Krusader
abika added a project to D22932: Remove 'isExiting' logic from closing sequence: Krusader.
Aug 4 2019, 5:25 PM · Krusader
abika requested changes to D22891: If an item that is going to be renamed is also going to be hidden by default, then warn the user.

I'm unsure about this. In contrast to Dolphin the target audience of Krusader are more advanced users. We should have more confidence in them imo. On the other hand a confirmation doesn't hurt...

Aug 4 2019, 4:03 PM · Krusader
abika accepted D22883: Clarify the aim of two lines written recently by David Faure.

Okay, I'm not against adding comments:)

Aug 4 2019, 3:40 PM · Krusader
abika added a comment to D22666: Achieve that Krusader searches correctly if the "Follow links" checkbox is disabled.

Ah, FileItem::isDir() can also return true for symlinks. Thanks again for fixing the bug i introduced, Toni!

Aug 4 2019, 3:28 PM · Krusader
abika added a comment to D22673: Set the focus on the first search result.

Hmm, when looking at my commit I cannot say what caused setting the "current" item before.
Anyway, thanks for fixing Toni!

Aug 4 2019, 3:04 PM · Krusader

Aug 2 2019

asensi requested review of D22891: If an item that is going to be renamed is also going to be hidden by default, then warn the user.
Aug 2 2019, 12:59 PM · Krusader

Aug 1 2019

asensi requested review of D22883: Clarify the aim of two lines written recently by David Faure.
Aug 1 2019, 9:50 PM · Krusader
asensi updated the diff for D15693: Shift+Del in order to remove cruft from the history of the "Search for" comboBox in the search dialog.

Minor changes were performed.

Aug 1 2019, 8:56 PM · Krusader
asensi added inline comments to D15693: Shift+Del in order to remove cruft from the history of the "Search for" comboBox in the search dialog.
Aug 1 2019, 8:52 PM · Krusader
asensi added a comment to D22673: Set the focus on the first search result.

I think the same, Nikita, and thanks for testing and informing!

Aug 1 2019, 8:41 PM · Krusader
asensi added a comment to D22666: Achieve that Krusader searches correctly if the "Follow links" checkbox is disabled.

I agree with you, Nikita. Thanks for testing and all the information!

Aug 1 2019, 8:39 PM · Krusader
asensi retitled D22666: Achieve that Krusader searches correctly if the "Follow links" checkbox is disabled from Achieve that Krusader searches correctly if the the "Follow links" checkbox is disabled to Achieve that Krusader searches correctly if the "Follow links" checkbox is disabled.
Aug 1 2019, 8:38 PM · Krusader
asensi closed D22673: Set the focus on the first search result.
Aug 1 2019, 5:56 AM · Krusader
asensi closed D22666: Achieve that Krusader searches correctly if the "Follow links" checkbox is disabled.
Aug 1 2019, 5:51 AM · Krusader

Jul 31 2019

nmel requested changes to D15693: Shift+Del in order to remove cruft from the history of the "Search for" comboBox in the search dialog.

It's a useful feature for people that like to keep their environment clean. Thanks for working on this, Toni!

Jul 31 2019, 5:07 PM · Krusader
nmel added a comment to D22666: Achieve that Krusader searches correctly if the "Follow links" checkbox is disabled.

If you'd like this fix to be included in v2.7.2, please push it to master by Aug 4 and I'll backport to stable. Please don't forget to specify BUG, FIXED, Differential Revision tags in the commit message.

Jul 31 2019, 4:25 PM · Krusader
nmel added a comment to D22673: Set the focus on the first search result.

If you'd like this fix to be included in v2.7.2, please push it to master by Aug 4 and I'll backport to stable. Please don't forget to specify BUG, FIXED, Differential Revision tags in the commit message.

Jul 31 2019, 4:25 PM · Krusader

Jul 30 2019

asensi updated the diff for D15693: Shift+Del in order to remove cruft from the history of the "Search for" comboBox in the search dialog.

The new files were updated (copyright years were changed).

Jul 30 2019, 8:48 AM · Krusader

Jul 29 2019

asensi updated the diff for D15693: Shift+Del in order to remove cruft from the history of the "Search for" comboBox in the search dialog.

This is the new version of the source code, after some minor improvements, having updated it because of the changes to the source code that have happened since the last time (like in krsearchdialog.cpp), and having used daily the classes meanwhile.

Jul 29 2019, 9:25 PM · Krusader

Jul 28 2019

nmel accepted D22666: Achieve that Krusader searches correctly if the "Follow links" checkbox is disabled.

It works properly now - thanks!
I checked the commit you mentioned in the bug. Before it the checkbox was still ignored but the links were always not followed.

Jul 28 2019, 6:48 AM · Krusader
nmel accepted D22673: Set the focus on the first search result.

It works and I'm fine with the proposed fix. Thanks Toni!

Jul 28 2019, 6:12 AM · Krusader

Jul 27 2019

asensi added reviewers for D22666: Achieve that Krusader searches correctly if the "Follow links" checkbox is disabled: gengisdave, nmel, abika, yurchor.
Jul 27 2019, 7:33 PM · Krusader
asensi added reviewers for D22673: Set the focus on the first search result: gengisdave, nmel, abika, yurchor.
Jul 27 2019, 7:30 PM · Krusader

Jul 23 2019

asensi accepted D22434: Replace obsolete QFontMetrics::width with QFontMetrics::horizontalAdvance.

(optional) I'd use QFontMetrics::horizontalAdvance(A) and QFontMetrics::width(A) for additional type check.

Jul 23 2019, 10:11 PM · Krusader