KrusaderProject
ActivePublic

Recent Activity

Today

alex added a comment to D27534: Utilize the building way used in other places like the 'Release Howto' and the INSTALL file.

what do you think about using make -j$(nproc) instead of just make this way all the available cores are used for compiling.
This is just a little trick I sometimes forget to change when I follow build instructions.

Fri, Feb 21, 9:06 AM · Krusader

Yesterday

asensi requested review of D27534: Utilize the building way used in other places like the 'Release Howto' and the INSTALL file.
Thu, Feb 20, 11:31 PM · Krusader

Wed, Feb 19

asensi accepted D27302: Update obsolete KProcess::error.

I've been using daily those changes for more than a week and everything looked alright. Other people can do their checks. Thanks, Davide!

Wed, Feb 19, 10:30 PM · Krusader
asensi accepted D27290: Update obsolete QWidget::getContentsMargins.

I've been using daily those changes for more than a week and everything looked alright. Other people can do their checks. Thanks, Davide!

Wed, Feb 19, 10:30 PM · Krusader
asensi accepted D27289: Update obsolete QString::sprintf.

I've been using daily those changes for more than a week and everything looked alright. Other people can do their checks. Thanks, Davide!

Wed, Feb 19, 10:29 PM · Krusader

Tue, Feb 18

gengisdave accepted D27446: Changes in names of variables, functions, and comments (due to recent changes and old ones).

Sounds good to me, no errors found

Tue, Feb 18, 7:16 AM · Krusader

Sun, Feb 16

asensi added a comment to D27259: Manage the use of Shift+F4 when opening a local file that exists, or a remote folder or remote file that exists.

Note: There are related changes in https://phabricator.kde.org/D27446

Sun, Feb 16, 9:38 PM · Krusader
asensi requested review of D27446: Changes in names of variables, functions, and comments (due to recent changes and old ones).
Sun, Feb 16, 9:30 PM · Krusader

Sat, Feb 15

asensi closed D27259: Manage the use of Shift+F4 when opening a local file that exists, or a remote folder or remote file that exists.

Thanks, Yuri and Davide!

Sat, Feb 15, 5:25 PM · Krusader
asensi closed D27335: Allow Krusader to be built under the 16.04 LTS versions of Ubuntu, Kubuntu, etc.
Sat, Feb 15, 5:15 PM · Documentation, Krusader
asensi closed D27258: Manage what happens when a PanelViewerBase (e.g. using Shift+F3) tries to open a nonexistent local file.

Closed by commit https://cgit.kde.org/krusader.git/commit/?id=419ac6ec839414bbae52849af536bc8ba1da83e4

Sat, Feb 15, 5:12 PM · Krusader
asensi added a comment to D27258: Manage what happens when a PanelViewerBase (e.g. using Shift+F3) tries to open a nonexistent local file.

Thanks, Yuri and Davide!

Sat, Feb 15, 5:01 PM · Krusader
asensi added a comment to D27335: Allow Krusader to be built under the 16.04 LTS versions of Ubuntu, Kubuntu, etc.

Thanks, Yuri, Antonio and Davide!

Sat, Feb 15, 5:00 PM · Documentation, Krusader
gengisdave accepted D27259: Manage the use of Shift+F4 when opening a local file that exists, or a remote folder or remote file that exists.

Compiled and tested on QT 5.14.1, QT 5.9.5 (Ubuntu 18.04) and QT 5.12.4 (Ubuntu 19.10); all fine

Sat, Feb 15, 11:24 AM · Krusader

Wed, Feb 12

gengisdave accepted D27335: Allow Krusader to be built under the 16.04 LTS versions of Ubuntu, Kubuntu, etc.

All good, thanks.

Wed, Feb 12, 10:16 AM · Documentation, Krusader
Abella accepted D27335: Allow Krusader to be built under the 16.04 LTS versions of Ubuntu, Kubuntu, etc.

Fine

Wed, Feb 12, 8:49 AM · Documentation, Krusader
yurchor accepted D27335: Allow Krusader to be built under the 16.04 LTS versions of Ubuntu, Kubuntu, etc.

No objections if it works.

Wed, Feb 12, 7:01 AM · Documentation, Krusader

Tue, Feb 11

asensi requested review of D27335: Allow Krusader to be built under the 16.04 LTS versions of Ubuntu, Kubuntu, etc.
Tue, Feb 11, 11:07 PM · Documentation, Krusader

Mon, Feb 10

gengisdave requested review of D27302: Update obsolete KProcess::error.
Mon, Feb 10, 6:58 PM · Krusader
gengisdave requested review of D27290: Update obsolete QWidget::getContentsMargins.
Mon, Feb 10, 3:19 PM · Krusader
gengisdave requested review of D27289: Update obsolete QString::sprintf.
Mon, Feb 10, 2:41 PM · Krusader
gengisdave closed D26325: Update deprecated KUrlLabel::leftClickedUrl.
Mon, Feb 10, 10:18 AM · Krusader
gengisdave closed D23924: Add QT::KeyMenu action to search results.
Mon, Feb 10, 10:13 AM · Krusader
gengisdave closed D26269: Remove deprecated mostLocalUrl.
Mon, Feb 10, 10:01 AM · Krusader
gengisdave accepted D27258: Manage what happens when a PanelViewerBase (e.g. using Shift+F3) tries to open a nonexistent local file.

All tests fine.

Mon, Feb 10, 9:47 AM · Krusader

Sat, Feb 8

yurchor added a comment to D27258: Manage what happens when a PanelViewerBase (e.g. using Shift+F3) tries to open a nonexistent local file.

Tested to work as expected.

Sat, Feb 8, 8:25 PM · Krusader
yurchor added a comment to D27259: Manage the use of Shift+F4 when opening a local file that exists, or a remote folder or remote file that exists.

Tested to work as expected.

Sat, Feb 8, 8:24 PM · Krusader
asensi requested review of D27259: Manage the use of Shift+F4 when opening a local file that exists, or a remote folder or remote file that exists.
Sat, Feb 8, 7:37 PM · Krusader
asensi requested review of D27258: Manage what happens when a PanelViewerBase (e.g. using Shift+F3) tries to open a nonexistent local file.
Sat, Feb 8, 7:16 PM · Krusader
gengisdave closed D26334: Select item from search window.
Sat, Feb 8, 1:52 PM · Krusader
gengisdave added a comment to D26334: Select item from search window.

Although the current fix works for me, i propose to change the added line into

panel->view->setNameToMakeCurrent(nameToMakeCurrent);
Sat, Feb 8, 1:50 PM · Krusader

Fri, Feb 7

asensi closed D27076: Update information in the INSTALL file, make some steps clearer, cover some more cases.
Fri, Feb 7, 11:47 PM · Krusader

Sat, Feb 1

yurchor accepted D27076: Update information in the INSTALL file, make some steps clearer, cover some more cases.

Thanks.

Sat, Feb 1, 7:59 AM · Krusader

Fri, Jan 31

asensi requested review of D27076: Update information in the INSTALL file, make some steps clearer, cover some more cases.
Fri, Jan 31, 10:10 PM · Krusader

Thu, Jan 30

asensi closed D24480: Change the document about helping Krusader, add other ways (2).

Closing it manually because it wasn't closed automatically when performing the commit.

Thu, Jan 30, 11:13 PM · Krusader
asensi closed D23309: Change the document about helping Krusader, add other ways.
Thu, Jan 30, 11:10 PM · Krusader, Documentation
asensi added a comment to D24480: Change the document about helping Krusader, add other ways (2).

This modification and the related one (https://phabricator.kde.org/D23309) can be applied after the latest changes by Yuri to the website (https://phabricator.kde.org/D26754). Thanks Pino, Yuri and Nikita!

Thu, Jan 30, 10:56 PM · Krusader
asensi added a comment to D23309: Change the document about helping Krusader, add other ways.

This modification and the related one (https://phabricator.kde.org/D24480) can be applied after the latest changes by Yuri to the website (https://phabricator.kde.org/D26754). Thanks Pino, Yuri and Nikita!

Thu, Jan 30, 10:50 PM · Krusader, Documentation

Mon, Jan 27

asensi accepted D24987: Add Kompare as compare application with KIO support.

The new version of the proposal keeps being acceptable, of course :-)

Mon, Jan 27, 11:22 PM · Krusader
asensi added a comment to D26291: rework of keyboard widgets focus.

So we have to consider that CTRL+UP/DOWN cannot be used anymore as before and users may complain about it. However, this did
also not work for the embedded terminal (see bug report) and for the sidebar before this patch.

Mon, Jan 27, 11:18 PM · Krusader, Documentation
asensi closed D26741: Synchronizer: Set the focus on the Compare button after selecting a profile.
Mon, Jan 27, 10:59 PM · Krusader
asensi updated the diff for D26741: Synchronizer: Set the focus on the Compare button after selecting a profile.

Previously, syncList->setFocus(); was utilized as it was used in three other places in the same file, although with btnCompareDirs->setFocus(); the intentions becomes clearer. Thanks, Alex! (and Yuri :-)

Mon, Jan 27, 10:57 PM · Krusader
abika accepted D26291: rework of keyboard widgets focus.

Good improvement in my opinion!
My test results:

  • The CTRL+SHIFT+UP/DOWN shortcuts work perfectly. Only when unfocusing the navigation bar a second time, the edit mode stays on. It might be a widget problem, but i consider it minor and it happened also before this patch.
  • Then i set CTRL+UP/DOWN as alternative shortcuts for the same actions to see if this can be used for backwards compatibility: This does not work when trying to unfocusing the navigation bar, the embedded terminal or the command line. All these widgets are capturing the key press and the shortcut action is not activated.
Mon, Jan 27, 5:46 PM · Krusader, Documentation
abika accepted D26334: Select item from search window.

Thanks for the fix Davide! This was also annoying me a lot.

Mon, Jan 27, 4:50 PM · Krusader
abika accepted D26741: Synchronizer: Set the focus on the Compare button after selecting a profile.

Thanks Toni. Works as expected.
But i still would recommend to either use btnCompareDirs->setFocus(); directly instead, or to add a short comment that syncList is focused to have the compare dirs button (which is set as default) being executed on pressing "Enter".
Otherwise it is a bit confusing why the empty tree widget is focused.

Mon, Jan 27, 3:10 PM · Krusader
abika added a comment to D24987: Add Kompare as compare application with KIO support.

I removed the two added brackets again.

Mon, Jan 27, 2:25 PM · Krusader
abika updated the diff for D24987: Add Kompare as compare application with KIO support.
  • fixup! Add Kompare as compare application with KIO support
Mon, Jan 27, 2:18 PM · Krusader
abika added a comment to D24809: Archive: Fix invalid destination URL when packing/unpacking from remote location.

[Got finally some time for Krusader again:)]

Mon, Jan 27, 2:08 PM · Krusader
Diffusion closed D24809: Archive: Fix invalid destination URL when packing/unpacking from remote location.
Mon, Jan 27, 2:05 PM · Krusader

Sat, Jan 25

asensi updated the diff for D24480: Change the document about helping Krusader, add other ways (2).

An address was changed, following what Yuri recently applied (e.g. it can be seen in Fix links on docs.kde.org).

Sat, Jan 25, 7:52 PM · Krusader