KrusaderProject
ActivePublic

Recent Activity

Yesterday

yurchor added a comment to D4008: A warning note, like the formerly used ones, was added to the installation section. .
In D4008#78221, @abika wrote:

I would not recommend the information of that page, a line like

su -c "make install"

won't work for Debian/Mint/Ubuntu/Kubuntu/etc. users.

Isn't sudo used for almost all distributions nowadays? We can replace it everywhere. And if somebody can compile Krusader he/she should know enough about Linux to get root access.)

Tue, Jan 17, 9:11 PM · Krusader, Websites
abika added a comment to D4008: A warning note, like the formerly used ones, was added to the installation section. .

I would not recommend the information of that page, a line like

su -c "make install"

won't work for Debian/Mint/Ubuntu/Kubuntu/etc. users.

Tue, Jan 17, 8:58 PM · Krusader, Websites
asensi added a comment to D4008: A warning note, like the formerly used ones, was added to the installation section. .

Thanks, there are solutions stated in the INSTALL file, I volunteer for adapting the web page :-)

Tue, Jan 17, 8:34 PM · Krusader, Websites
yurchor added a comment to D4008: A warning note, like the formerly used ones, was added to the installation section. .
In D4008#78168, @asensi wrote:

Hi!

the current instructions (5 commands) should work well for most users.

I would not recommend the information of that page, a line like

su -c "make install"

won't work for Debian/Mint/Ubuntu/Kubuntu/etc. users.
a line like

wget http://websvn.kde.org/*checkout*/trunk/l10n-kf5/de/messages/extragear-utils/krusader.po

does not work, it writes "ERROR 403: Forbidden.". The line

su -c "cp krusader.mo /usr/share/locale/de/LC_MESSAGES/"

misleads most users (because of the "su -c" and because of the "de"), and there were some other problems that led Davide and me to the improvement of the INSTALL file.

Tue, Jan 17, 8:21 PM · Krusader, Websites
asensi added a comment to D4008: A warning note, like the formerly used ones, was added to the installation section. .

the current instructions (5 commands) should work well for most users.

Tue, Jan 17, 7:59 PM · Krusader, Websites
abika added a comment to D4008: A warning note, like the formerly used ones, was added to the installation section. .

The idea was to follow the Krusader handbook style (in docs.kde.org) to put out a big warning, in this case: "do not follow the next outdated information, it will cause you problems", "follow the linked file" :-)

Tue, Jan 17, 3:52 PM · Krusader, Websites
bcooksley removed a member for Krusader: bcooksley.
Tue, Jan 17, 7:40 AM
asensi added a comment to D4008: A warning note, like the formerly used ones, was added to the installation section. .

But why a single (big) one with different style?

Tue, Jan 17, 7:22 AM · Krusader, Websites

Mon, Jan 16

abika requested changes to D4008: A warning note, like the formerly used ones, was added to the installation section. .

Hmm, sorry I don't think the different style fits to the rest of the page. There are already a lot of Note:s . We should change all of them. But why a single (big) one with different style?

Mon, Jan 16, 2:46 PM · Krusader, Websites
asensi added a comment to D4008: A warning note, like the formerly used ones, was added to the installation section. .

I understand the motivation behind the additional css now, so I'm also fine with your proposed changes.

Mon, Jan 16, 12:37 AM · Krusader, Websites

Sun, Jan 15

abika added a comment to D3994: add asynchronous vfs refresh.

I'm actually unsure if there is any benefit for a "real" async refresh. The panel currently depends on it and the Qt event loop prevents an UI freeze during a long refresh.

Sun, Jan 15, 9:45 PM · Krusader
abika added a comment to T5057: Check usage of QDir::setCurrent().

I have no idea why the directory change is there in the code or why it could be important. Maybe remove it and see what happens(?)

Sun, Jan 15, 9:41 PM · Krusader
janlepper updated the summary for D4146: ListPanel: don't go back in history if refresh failed.
Sun, Jan 15, 8:46 PM · Krusader
martinkostolny accepted D4146: ListPanel: don't go back in history if refresh failed.

I'm definitely for this change! I don't like loosing the path when e.g. external disk is temporarily removed. Thanks!

Sun, Jan 15, 8:40 PM · Krusader
janlepper added a task to D4146: ListPanel: don't go back in history if refresh failed: T5017: Avoid nested event dispatching.
Sun, Jan 15, 8:32 PM · Krusader
janlepper added a revision to T5017: Avoid nested event dispatching: D4146: ListPanel: don't go back in history if refresh failed.
Sun, Jan 15, 8:32 PM · Krusader
janlepper added a reviewer for D4146: ListPanel: don't go back in history if refresh failed: Krusader.
Sun, Jan 15, 7:53 PM · Krusader
janlepper updated the summary for D4146: ListPanel: don't go back in history if refresh failed.
Sun, Jan 15, 7:52 PM · Krusader
martinkostolny accepted D4008: A warning note, like the formerly used ones, was added to the installation section. .

I understand the motivation behind the additional css now, so I'm also fine with your proposed changes. We can always adjust it later if we change our minds :).

Sun, Jan 15, 3:35 PM · Krusader, Websites

Thu, Jan 12

andreaska added a comment to T5062: Improve search results tab.

yes

Thu, Jan 12, 1:57 PM · Krusader
janlepper added a comment to T5062: Improve search results tab.

How about that?

Thu, Jan 12, 1:43 PM · Krusader
andreaska added a comment to T5062: Improve search results tab.

when it's a mockup I would suggest to have an overview on the right where you see the results in the file.

Thu, Jan 12, 1:30 PM · Krusader
janlepper added a comment to T5062: Improve search results tab.

I'm sorry - it's just a mockup ;)
I don't think I'll be working on this in the near future - just wanted to collect some ideas.
The mc-like result list will probably require a lot of work - in KrView there is a 1:1 relationship between view item and a file, but this would require multiple items per file.

Thu, Jan 12, 1:27 PM · Krusader
janlepper edited the description of T5062: Improve search results tab.
Thu, Jan 12, 1:16 PM · Krusader

Wed, Jan 11

ltoscano added a comment to D3994: add asynchronous vfs refresh.

Please don't restrict the reviews!

Wed, Jan 11, 10:01 PM · Krusader
bcooksley changed the edit policy for D3994: add asynchronous vfs refresh.
Wed, Jan 11, 9:59 PM · Krusader
bcooksley added a member for Krusader: bcooksley.
Wed, Jan 11, 9:57 PM
abika added a comment to T5062: Improve search results tab.

Great! Nice feature. Is that KWrite from KParts?

Wed, Jan 11, 5:30 PM · Krusader
janlepper edited the description of T5062: Improve search results tab.
Wed, Jan 11, 4:56 PM · Krusader
janlepper edited the description of T5062: Improve search results tab.
Wed, Jan 11, 4:40 PM · Krusader
janlepper created T5062: Improve search results tab.
Wed, Jan 11, 4:37 PM · Krusader
janlepper added a comment to D3994: add asynchronous vfs refresh.

Unfortunately I don't have this area studied enough yet to consult the direction. But I certainly can test your code and report issues:). So far I found some regarding refreshing:

  • external changes aren't reflected (I have to manual refresh Ctrl+R to see them)
  • undo action will not refresh view after it is done

    But I understand it is a work in progress, so please say if it is too soon to test these "corner" cases. Anyway, thanks for Your work!
Wed, Jan 11, 9:59 AM · Krusader

Tue, Jan 10

asensi added a comment to D4008: A warning note, like the formerly used ones, was added to the installation section. .

Hi, Martin, the changes that you propose are good. The css file was added to achieve that the note:

Note
Please read the INSTALL file of the Krusader sources, it contains all latest installation instructions.

was seen the same way as the same notes that are seen in the Krusader Handbook (in docs.kde.org), for example in: https://docs.kde.org/trunk5/en/extragear-utils/krusader/kde_lin_install.html , https://docs.kde.org/trunk5/en/extragear-utils/krusader/compilation.html , etc.

Tue, Jan 10, 11:15 PM · Krusader, Websites
martinkostolny added a comment to D3994: add asynchronous vfs refresh.

Unfortunately I don't have this area studied enough yet to consult the direction. But I certainly can test your code and report issues:). So far I found some regarding refreshing:

  • external changes aren't reflected (I have to manual refresh Ctrl+R to see them)
  • undo action will not refresh view after it is done
Tue, Jan 10, 9:44 PM · Krusader
martinkostolny added a comment to D4008: A warning note, like the formerly used ones, was added to the installation section. .

I like the idea about the installation note! But is it necessary to pull another css? How about leaving a plain paragraph to visually align with another paragraphs under rest of headings? Something like:

Tue, Jan 10, 8:50 PM · Krusader, Websites
janlepper added a comment to T5057: Check usage of QDir::setCurrent().

The invocation In ListPanelFunc::doRefresh() seems to rely on the panel being the active one (which was true historically - when refreshing a panel would activate itself first).

Tue, Jan 10, 7:03 PM · Krusader
janlepper created T5057: Check usage of QDir::setCurrent().
Tue, Jan 10, 6:58 PM · Krusader
janlepper removed a dependent revision for D3994: add asynchronous vfs refresh: Restricted Differential Revision.
Tue, Jan 10, 2:37 AM · Krusader
janlepper set the repository for D3994: add asynchronous vfs refresh to R167 Krusader.
Tue, Jan 10, 2:23 AM · Krusader
janlepper updated the diff for D3994: add asynchronous vfs refresh.
Tue, Jan 10, 2:23 AM · Krusader
janlepper added a dependent revision for D3994: add asynchronous vfs refresh: Restricted Differential Revision.
Tue, Jan 10, 12:38 AM · Krusader

Mon, Jan 9

janlepper changed the visibility for D3994: add asynchronous vfs refresh.
Mon, Jan 9, 9:11 PM · Krusader

Sun, Jan 8

janlepper added a reviewer for D3994: add asynchronous vfs refresh: Krusader.
Sun, Jan 8, 5:11 PM · Krusader
asensi created D4008: A warning note, like the formerly used ones, was added to the installation section. .
Sun, Jan 8, 9:36 AM · Krusader, Websites

Sat, Jan 7

abika added a comment to T5038: Update documentation links in order to point to the new Krusader 2.5.0 handbook.

Sorry Tonny. It just happened that we worked on it at the same time.

Sat, Jan 7, 8:39 PM · Krusader
asensi closed T5038: Update documentation links in order to point to the new Krusader 2.5.0 handbook as "Resolved".

Alex Bikadorov solved it :-)

Sat, Jan 7, 7:07 PM · Krusader
asensi abandoned D4004: Update documentation links in order to point to the new Krusader 2.5.0 handbook.

Alex Bikadorov solved it :-)

Sat, Jan 7, 7:06 PM · Krusader
yurchor accepted D4004: Update documentation links in order to point to the new Krusader 2.5.0 handbook.
Sat, Jan 7, 6:28 PM · Krusader
abika added a comment to T5038: Update documentation links in order to point to the new Krusader 2.5.0 handbook.

... or better: I already did. Check the website.

Sat, Jan 7, 6:20 PM · Krusader
abika added a comment to T5038: Update documentation links in order to point to the new Krusader 2.5.0 handbook.

I'm currently doing this :/

Sat, Jan 7, 6:20 PM · Krusader