D12558: Don't filter out duplicated entries from places panel
Elvis Angelaccio
noreply at phabricator.kde.org
Sun Apr 29 12:03:09 BST 2018
elvisangelaccio accepted this revision as: elvisangelaccio.
elvisangelaccio added a comment.
This revision is now accepted and ready to land.
This seems indeed an unrelated change from D9985 <https://phabricator.kde.org/D9985>. Since that commit doesn't explain why this code was needed, I think we can drop it.
In D12558#254797 <https://phabricator.kde.org/D12558#254797>, @ngraham wrote:
> However is this what we want, behaviorally? The code you're removing was designed to prevent showing duplicates.
Nope, that is handled by D9333 <https://phabricator.kde.org/D9333> and it should still work after this patch.
REPOSITORY
R318 Dolphin
REVISION DETAIL
https://phabricator.kde.org/D12558
To: arojas, #dolphin, ngraham, elvisangelaccio, renatoo
Cc: xyquadrat
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180429/51600558/attachment.htm>
More information about the kfm-devel
mailing list