| Summary: | Dolphin can't decide how to sort some files; changes its mind | ||
|---|---|---|---|
| Product: | [Applications] dolphin | Reporter: | Brendon Higgins <brendon> |
| Component: | general | Assignee: | Peter Penz <peter.penz19> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | bugzilla, frank78ac, vortex |
| Priority: | NOR | Keywords: | reproducible |
| Version: | 16.12.2 | ||
| Target Milestone: | --- | ||
| Platform: | unspecified | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | 4.5.3 | |
| Attachments: |
screencast showing "jumpy" sorting in dolphin
screenshot showing erratic sort order in dolphin |
||
|
Description
Brendon Higgins
2010-05-16 07:05:05 UTC
Thanks for the bug report! Probably related to 237621/bug 217048. I have the same problem: when files are named in a specific way (I haven't figured out yet, what exactly is the cause) they jump around when you mouse-over them. If you disable thumbnail previews, files do not jump around anymore when hovered. However, they will be sorted in wrong order. If you turn off "Natural sorting of items" the problems vanish. Created attachment 43869 [details]
screencast showing "jumpy" sorting in dolphin
This screencast shows items in a folder change place when you hover them. It happens when previews and natural sort order are enabled.
Another example screencast, showing a folder of images in icon preview mode: http://www.mediafire.com/?ykykowbgwmd Created attachment 43870 [details]
screenshot showing erratic sort order in dolphin
This shows the folder from the last example with previews turned off.
Files do not change places anymore when hovered, but they are sorted in wrong order.
This is a duplicate of bug 217048 and solved already (it has been backported to KDE SC 4.4.4). *** This bug has been marked as a duplicate of bug 217048 *** @Benjamin: Thanks for the screenshots! Although in this case the issue was resolved already, usually this helps us a lot. Problem still exists in KDE SC 4.5.1. @Janet: I cannot reproduce this issue with 4.5.1 I can reproduce it in KDE 4.5.1, just was about to file a bug report about the issue, there are several things that must match: 1. natural sorting must be enabled! 2. previews must be enabled! 3. you need at least three files that start with the same letters in the name 4. at least one of the files has an underscore right after those same letters, one has a number (not zero!) after those letters and one has the file ending right after those letters, e.g. abc_a.jpg, abc.jpg and abc1.jpg - three files are the minimum for the show. It can take up some seconds before the files begin to switch positions. Usually hovering and staying over the last of the three files for a while starts the process. I also have brought abc.txt, abc_a.txt and abc1.txt to "dance", but only once. It doesn't happen always, it is way easier to demonstrate it with pictures, they tend to dance more than text files, guess it has to do something with the thumbnail. So try using photos and rename them accordingly. Thanks Janet for the detailed instructions. I could reproduce the issue in trunk. SVN commit 1188178 by ppenz: Revert the patch to handle ' ', '.' and '_' equally, as this violates the rule: if a < b and b < c, a must also be < c. The issue can be reproduced with the strings abc.jpg, abc1.jpg and abc_a.jpg. I really propose to make this "natural sorting" not more clever than necessary, just handling digits in a "natural" way is sufficient from my point of view. BUG: 237788 FIXED-IN: 4.5.3 M +8 -8 tests/kstringhandlertest.cpp M +5 -8 text/kstringhandler.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1188178 SVN commit 1188180 by ppenz: Forward port: Revert the patch to handle ' ', '.' and '_' equally, as this violates the rule: if a < b and b < c, a must also be < c. The issue can be reproduced with the strings abc.jpg, abc1.jpg and abc_a.jpg. I really propose to make this "natural sorting" not more clever than necessary, just handling digits in a "natural" way is sufficient from my point of view. CCBUG: 237788 M +8 -8 tests/kstringhandlertest.cpp M +5 -8 text/kstringhandler.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1188180 Thanks Janet, Benjamin and Brendon for the good descriptions, I could reproduce the issue now and have applied a fix. I've reverted a patch that was written to improve the natural sorting (e. g. "Episode 24" should be smaller than "Episode_25"), however this resulted in unpredictable orders. I'll take care in future to not accept such kind of patches for natural sorting and just define "natural sorting" as handling digits in a special way. I've added this bug as part of the unit-tests to detect possible regressions in future. Explains a lot. No signs should be treated equal, this always will lead to problems in sorting. I fully agree, natural sorting should only concern digits, no other signs. Thanks a LOT for reverting this! What I don't understand: why did this only happen when previews are turned on? > What I don't understand: why did this only happen when previews are turned on?
When previews are turned on, the items are updated asynchronously and trigger a different comparison order (it just compares the items that require an update). As it was not assured that if a<b and b<c that also a<c, this resulted in this "jumping". I guess that with your example even without previews the order of the items was non-deterministic, but at least no jumping happened. But I'm glad that this is solved now :-)
|