I would not recommend the information of that page, a line likesu -c "make install"
won't work for Debian/Mint/Ubuntu/Kubuntu/etc. users.
Thanks, there are solutions stated in the INSTALL file, I volunteer for adapting the web page :-)
the current instructions (5 commands) should work well for most users.
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" :-)
But why a single (big) one with different style?
Mon, Jan 16
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?
I understand the motivation behind the additional css now, so I'm also fine with your proposed changes.
Sun, Jan 15
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.
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(?)
I'm definitely for this change! I don't like loosing the path when e.g. external disk is temporarily removed. Thanks!
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 :).
Thu, Jan 12
How about that?
when it's a mockup I would suggest to have an overview on the right where you see the results in the file.
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.
Wed, Jan 11
Please don't restrict the reviews!
Great! Nice feature. Is that KWrite from KParts?
Tue, Jan 10
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.
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
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:
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).
Mon, Jan 9
Sun, Jan 8
Sat, Jan 7
Sorry Tonny. It just happened that we worked on it at the same time.
Alex Bikadorov solved it :-)
Alex Bikadorov solved it :-)
... or better: I already did. Check the website.
I'm currently doing this :/