In D17595#473825, @meven wrote:I guess we need now to reach to schwarzerf@gmail.com romaninflianskas mfuchs chinmoyr mitchell emateli emmanuelp freininghaus to have lgplv2+ approval.
- Queries
- All Stories
- Search
- Advanced Search
Feed Advanced Search
Advanced Search
Advanced Search
Jun 11 2019
Jun 11 2019
emmanuelp committed R230:e1dc8b07716e: Add myself (emmanuelp) to relicensecheck.pl (authored by emmanuelp).
Add myself (emmanuelp) to relicensecheck.pl
Dec 4 2017
Dec 4 2017
You shouldn't depend on the implementation details of ioslaves. KIO has a well-defined interface to determine if an protocol can be handled.
And if the implementation of this interface is not good enough, then it needs to be improved (e.g. decrease number of file system look-ups, better cache invalidation, ...).
Oct 14 2017
Oct 14 2017
Sep 21 2017
Sep 21 2017
emmanuelp committed R318:3749b5552729: Select child folder when navigating to a parent folder (authored by gregormi).
Select child folder when navigating to a parent folder
A better fix is https://git.reviewboard.kde.org/r/123253/
Sep 15 2017
Sep 15 2017
Sep 12 2017
Sep 12 2017
In D7700#144819, @nicolasfella wrote:What do you think would be a reasonable number of tags? 5, 10, 20, ..? Make it configurable?
Sep 11 2017
Sep 11 2017
(I'll commit your patches in the evening)
Thanks for the patch!
In D7690#144674, @ngraham wrote:@emmanuelp Does this look commitable now?
emmanuelp requested changes to D7696: Add "Width" and "Height" separately to Additional Information -> Images.
In D7440#138187, @broulik wrote:-1 Urgh. Can we perhaps have a static QStringList KIO::PreviewJob::defaultPlugins()? It's not just Dolphin that has a copy of this (file dialog et al). And this way we could have flip it around to whitelist all plugins (find them through KService) and then have a user blacklist and perhaps by default blacklist stupid thumbnailers like "plain text".
In D7655#143872, @elvisangelaccio wrote:
- This patch.
Sep 6 2017
Sep 6 2017
In D7571#142637, @dfaure wrote:An application copying some config file in the background (or any other internal file operation) shouldn't bother the user with prompts.
Hmm I don't think that we should add additional shortcuts to user-defined shortcuts. What if the user assigned {Shift, Del} to something different?
Some nitpicks, looks good otherwise!
Sep 3 2017
Sep 3 2017
Sep 1 2017
Sep 1 2017
emmanuelp committed R318:94fab8c80ed6: Limit folder panel to home directory if inside home (authored by hoffmannrobert).
Limit folder panel to home directory if inside home
emmanuelp closed D7477: Limit folder panel to home directory if inside home by committing R318:94fab8c80ed6: Limit folder panel to home directory if inside home (authored by hoffmannrobert).
Nice work! Thanks!
Aug 31 2017
Aug 31 2017
Last round
Aug 30 2017
Aug 30 2017
Aug 29 2017
Aug 29 2017
Nice work! :)
Aug 25 2017
Aug 25 2017
LGTM!
Nice work!
emmanuelp added a comment to D7390: Support middle clicking of Back/Forward/Up/Home toolbar buttons.
Nice addition :)
Jun 20 2017
Jun 20 2017
In D6269#117847, @broulik wrote:
- Adjust timeRoleGroups so it takes a lambda where we can then return the proper timestamp (either from KFileItem or a role)
Special handling of DeletionDateRole is missing in:
- KFileItemModel::sortRoleCompare (see ModificationTimeRole for an example)
- KFileItemModel::groups -> requires that timeRoleGroups() uses the sort role instead of KFileItem::FileTimes
In D6269#117553, @cfeck wrote:If trash KIO is already inconsistent we can indeed pick being inconsistent with either it or with the rest of Dolphin.
But I am concerned about the UI string.
Mar 30 2017
Mar 30 2017
Oh thanks! Totally missed it during review :/
emmanuelp added a comment to D4134: Take into account protocol class when determining whether it's a local file for preview.
In D4134#98027, @broulik wrote:Given PreviewJob now does the "right thing" by default, I think we can discard this.
Mar 23 2017
Mar 23 2017
emmanuelp added a comment to D5138: Fill UDS_CREATION_TIME with the value of st_birthtime on FreeBSD.
In D5138#97024, @adridg wrote:So, Linux, sort out your *stat(2) interfaces first and we'll get back to you.
Mar 14 2017
Mar 14 2017
Mar 13 2017
Mar 13 2017
emmanuelp renamed T5599: [Junior Job] Add 'Genre' to additional information from Add 'Genre' to additional information to [Junior Job] Add 'Genre' to additional information.
Mar 8 2017
Mar 8 2017
Mar 7 2017
Mar 7 2017
Nice work! :)
Mar 5 2017
Mar 5 2017
Nice feature :)
Nice, thanks!
Can you please add a new function to global.h which encapsulates all this?
Maybe "Dolphin::openNewWindow(urls, flags = None)"
Feb 23 2017
Feb 23 2017
This is a test notification, sent at Thu, Feb 23, 16:04.
emmanuelp accepted D4560: Increase smooth scrolling animation duration from 100 to 300 ms and set easing curve to InOutQuart.
Do you have commit access?
Feb 22 2017
Feb 22 2017
emmanuelp added a comment to D4560: Increase smooth scrolling animation duration from 100 to 300 ms and set easing curve to InOutQuart.
In D4560#87675, @vishalrao wrote:I guess, yeah 300 ms and InOutQuart would look good to me too.
Feb 19 2017
Feb 19 2017
emmanuelp committed R318:0bdd8e0b0516: Disallow executing Dolphin as root on Linux (authored by emmanuelp).
Disallow executing Dolphin as root on Linux
Feb 18 2017
Feb 18 2017
emmanuelp added a comment to D4560: Increase smooth scrolling animation duration from 100 to 300 ms and set easing curve to InOutQuart.
Thanks for the patch!
Add a gitignore file
Added .kdev4 to gitignore
emmanuelp committed R318:edf8e5737316: Make "show filter bar" a toggle action (authored by donnguyen).
Make "show filter bar" a toggle action
emmanuelp committed R318:db3f3746798a: Apply the file preview shadow frame to most previews instead of only image file… (authored by Diego Soenens <hyperz.2007@gmail.com>).
Apply the file preview shadow frame to most previews instead of only image file…
emmanuelp committed R318:6b727613db73: Remove the preserving of the selected items in reload(), this is already… (authored by emmanuelp).
Remove the preserving of the selected items in reload(), this is already…
emmanuelp committed R318:41b0e4297313: Restore the view state after the URL of the DolphinView has been changed, as… (authored by emmanuelp).
Restore the view state after the URL of the DolphinView has been changed, as…
emmanuelp committed R318:1d0b6ba65ff6: Start an anchored selection when restoring the selected items to make it… (authored by emmanuelp).
Start an anchored selection when restoring the selected items to make it…
emmanuelp committed R318:8bc6d09c57eb: Preserve selected items when changing folders (authored by emmanuelp).
Preserve selected items when changing folders
Feb 11 2017
Feb 11 2017
Feb 8 2017
Feb 8 2017
Nice!
Thanks for the adjustments, looks good!
Feb 5 2017
Feb 5 2017
Looks good otherwise!
Nice, thanks!
emmanuelp added a comment to D4134: Take into account protocol class when determining whether it's a local file for preview.
I'm wondering if we should make this the default in PreviewJob
Jan 31 2017
Jan 31 2017
Ah ok submitting my previous comment fixed it (don't know what happened) :)
The diff is somehow corrupted ...
Jan 14 2017
Jan 14 2017
I see some potential problems with the QPalette key approach. The key only covers some parts of the palette, so it is not really future proof when colors different than text, highlight or highlighted text are used in future. It also requires more memory because the cache still holds old unused pixmaps (only true if number of used pixmaps doesn't exceed the cache limit). Also the keys are much longer which requires more work when generating and hashing the keys.
Jan 12 2017
Jan 12 2017
Wouldn't it be easier if you just clear the pixmap cache on palette change? e.g. in MainWindow?
A palette change basically invalidates all cached icons, so I don't see any advantage in adding the palette information to the key (... use cases like frequently toggling between two palettes are not supported).
Jan 1 2017
Jan 1 2017