In D10832#613043, @meven wrote:ping @rominf
It would probably need some rebasing after so long....
- Queries
- All Stories
- Search
- Advanced Search
Feed Advanced Search
Advanced Search
Advanced Search
Feb 17 2020
Feb 17 2020
Feb 23 2019
Feb 23 2019
Looks good for me!
Feb 22 2019
Feb 22 2019
In D11382#417116, @ngraham wrote:@rominf, are you planning to come back and work on this at all?
Apr 13 2018
Apr 13 2018
rominf committed R318:ec12391a1bb8: Convert hard coded shortcuts to standard keys (authored by rominf).
Convert hard coded shortcuts to standard keys
Mar 29 2018
Mar 29 2018
rominf committed R241:f1ab805c4ebc: Fix build on FreeBSD broken by d7cce9937d5e9af2753fadb82d11f308b58bb8fa (authored by rominf).
Fix build on FreeBSD broken by d7cce9937d5e9af2753fadb82d11f308b58bb8fa
Mar 27 2018
Mar 27 2018
rominf updated the diff for D11716: Fix build on FreeBSD broken by d7cce9937d5e9af2753fadb82d11f308b58bb8fa.
- Replace #ifdef with #if
Mar 26 2018
Mar 26 2018
rominf updated the diff for D11716: Fix build on FreeBSD broken by d7cce9937d5e9af2753fadb82d11f308b58bb8fa.
- Consider criticism
rominf updated the diff for D11716: Fix build on FreeBSD broken by d7cce9937d5e9af2753fadb82d11f308b58bb8fa.
- Add a trailing newline to the end of the file
rominf updated the diff for D11716: Fix build on FreeBSD broken by d7cce9937d5e9af2753fadb82d11f308b58bb8fa.
- Actualy fixing build
rominf requested review of D11716: Fix build on FreeBSD broken by d7cce9937d5e9af2753fadb82d11f308b58bb8fa.
Support NTFS hidden files
Mar 25 2018
Mar 25 2018
- Consider criticism
- Split multiple statements to put one statement on a line
- Consider criticism
Mar 22 2018
Mar 22 2018
rominf updated the diff for D10960: Ask for confirmation when Closing Dolphin windows with a terminal panel running a program.
- Revert setDefault to the Yes button when closing multiple tabs
Mar 18 2018
Mar 18 2018
In D10959#228679, @elvisangelaccio wrote:Am I wrong of defocus does not work? Did we agree to ignore it?
In D10990#228540, @rkflx wrote:In D10990#228470, @elvisangelaccio wrote:Thanks, remember to use a descriptive commit message :)
@rominf If Elvis is hinting at something but you don't change anything, you should think twice before committing. This should have merely referred to the other commit with the same name and included the fact that here you are doing a follow-up commit to fix problems of the original commit.
Please respect that for future patches. Those requirements are not there to annoy you, but to keep the Git history usable for everyone working on Dolphin now and in the future.
I commited my changes and then landed them with:
git commit -am 'Rename "Show Original" -> "Show Target"' arc land --revision D10990
rominf added inline comments to D11334: Disable Create New menu and button when Trash dir is active.
rominf committed R318:6f05c66cc0d2: Add "Show Target" into symlink context menu and file menu (authored by rominf).
Add "Show Target" into symlink context menu and file menu
- focusTerminal -> focusTerminalPanel
- "Focus Terminal" -> "Focus Terminal Panel"
- Take criticism into account
- Use semantic tag filename in warning message
rominf committed R318:eb68fae15cdb: Show "Empty Trash" button inside trash directory (authored by rominf).
Show "Empty Trash" button inside trash directory
Mar 16 2018
Mar 16 2018
rominf added a reviewer for D11382: Add an option to show tabs from last time when Dolphin starts: VDG.
In D11382#227082, @broulik wrote:How does that behave when I start Dolphin a second time? I'm a person that usually has 5 Dolphin windows open with tons of tabs inside of them.
It opens a new instance of Dolphin identical to the previous one... I guess that's a wrong behavior. Should the new instance be fresh if there are other Dolphins running? If so, how to check this? DBus?
rominf updated the test plan for D11382: Add an option to show tabs from last time when Dolphin starts.
rominf updated the test plan for D11382: Add an option to show tabs from last time when Dolphin starts.
rominf updated the test plan for D11382: Add an option to show tabs from last time when Dolphin starts.
- m_emptyTrashButton->show/hide -> m_emptyTrashButton->setVisible
- Bump dolphinui.rc version
Mar 15 2018
Mar 15 2018
OK, the main problem from your comments is that it's enabled by default. I followed the example of Okular with it's hjkl keys for movements (those keys are enabled by default, you can test it).
Lot's of actions can also be implemented. I'm looking for a feedback. If feedback will be positive, I'll implement additional functionality like switching between tabs.
- Take @elvisangelaccio criticism into account
rominf added inline comments to D11334: Disable Create New menu and button when Trash dir is active.
- Remove unnecessary line
Mar 14 2018
Mar 14 2018
rominf committed R318:86f4fcb8cd57: Add "Open symlink destination folder" into symlink's context menu (authored by rominf).
Add "Open symlink destination folder" into symlink's context menu
- Disable if multiple files are selected
- Add "Show Original" to file menu
- Close action: "Close" -> "Close Tab"
In D10804#223457, @markg wrote:In D10804#223410, @rominf wrote:In D10804#223394, @markg wrote:@rominf if i restore from within the trash (trash is empty after restoring), the trash icon in the places menu and the "Empty Trash" button both stay active.
Could you take a look at that?It works for me. Can you check it again (I rebased this on master).
Just done that, same as before.
The icon only updates if i click on the trash again.
Still cannot reproduce. @elvisangelaccio, @ngraham Can you ckeck it?
- Cleanup
- Move isNtfsHidden to file ioslave
rominf added a comment to D10960: Ask for confirmation when Closing Dolphin windows with a terminal panel running a program.
In D10960#225162, @ngraham wrote:For the confirmation dialog added here, I'm unsure whether we should follow the advice of our inline conversation and make the Quit button be default. Depending on what's running in the Terminal panel, killing it could be quite destructive. But I won't block on that, and will accept whatever Dolphin maintainers decide.
I agree with @ngraham (I also for Cancel as a default). @elvisangelaccio what do you think?
@ngraham OK, no problem.
Mar 13 2018
Mar 13 2018
rominf updated the diff for D11293: Use Cancel button by default when closing Dolphin with multiple tabs.
- Remove unused line
rominf added inline comments to D10960: Ask for confirmation when Closing Dolphin windows with a terminal panel running a program.
rominf updated the diff for D10960: Ask for confirmation when Closing Dolphin windows with a terminal panel running a program.
- Remove unrelated changes
rominf requested review of D11293: Use Cancel button by default when closing Dolphin with multiple tabs.
rominf added inline comments to D10960: Ask for confirmation when Closing Dolphin windows with a terminal panel running a program.
rominf updated the diff for D10960: Ask for confirmation when Closing Dolphin windows with a terminal panel running a program.
- Set Cancel active by default when closing Dolphin with multiple tabs or running
program in terminal panel
Please review
Mar 12 2018
Mar 12 2018
In D10832#223393, @elvisangelaccio wrote:There is no reason to rush this in now, I'd suggest to wait the feature freeze on March 22 [1] so that we have enough time to be sure we don't break anything.
[1]: https://community.kde.org/Schedules/Applications/18.04_Release_Schedule
In D10832#223379, @markg wrote:Large review!
I was going to give a ship it till i compile tested it. It failed!
You can find the output here: https://p.sc2.nl/HJDBOJmtM
rominf added inline comments to D10960: Ask for confirmation when Closing Dolphin windows with a terminal panel running a program.
Mar 11 2018
Mar 11 2018
In D10804#223394, @markg wrote:@rominf if i restore from within the trash (trash is empty after restoring), the trash icon in the places menu and the "Empty Trash" button both stay active.
Could you take a look at that?
It works for me. Can you check it again (I rebased this on master).
- Rebase on master
rominf committed R318:68a924af05e7: Show a message if Konsole part is not installed (authored by rominf).
Show a message if Konsole part is not installed
- Take criticism into account
- Take criticism into account
Please review.
- Trash::instance.*empty -> Trash::*empty
Update trash icon after restore operation
- Rebase on master
- Add this as receiver argument
rominf removed a dependent revision for D11216: Update trash icon after restore operation: D10804: Show "Empty Trash" button inside trash directory.
rominf removed a dependency for D10804: Show "Empty Trash" button inside trash directory: D11216: Update trash icon after restore operation.
- Rebase on master
- Rebase on master
rominf committed R318:f16fbcba4e9e: Add Trash (empty, isEmpty, emptinessChanged) (authored by rominf).
Add Trash (empty, isEmpty, emptinessChanged)
- Trash::instance.*empty -> Trash::*empty
- Accept feedback
- Show a message about reopening the terminal panel instead of polling
rominf edited dependencies for D10804: Show "Empty Trash" button inside trash directory, added: 1; removed: 1.
rominf added a dependent revision for D11216: Update trash icon after restore operation: D10804: Show "Empty Trash" button inside trash directory.
rominf removed a dependent revision for D11012: Add Trash (empty, isEmpty, emptinessChanged): D10804: Show "Empty Trash" button inside trash directory.