Summary: | dolphin rename file selects part of the extension when name contains composed glyphs (diacritics, emojis) | ||
---|---|---|---|
Product: | [Applications] dolphin | Reporter: | kdebugs |
Component: | view-engine: general | Assignee: | Dolphin Bug Assignee <dolphin-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | kde, kfm-devel, lingm+kdebugs |
Priority: | NOR | ||
Version: | 22.08.2 | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Other | ||
Latest Commit: | https://invent.kde.org/system/dolphin/-/commit/34d3f770bcc5759f46b0ccc8fcad9956c7fbed0a | Version Fixed In: | |
Sentry Crash Report: |
Description
kdebugs
2023-03-04 07:52:09 UTC
I also just experienced this bug. Dolphin version 23.04.0. The amount of open bugs around dolphin just shocked me! A possibly relevant merge request was started @ https://invent.kde.org/system/dolphin/-/merge_requests/809 Git commit 021365dceb590a14bfcdb904ea05ffdd69b7d663 by Yifan Zhu. Committed on 13/08/2024 at 16:09. Pushed by fanzhuyifan into branch 'master'. KStandardItemListWidget: select by number of unicode chars Previously during rename, the number of QChar is used for selection, which might be different from number of unicode characters. Test plan: - create the file zz❤️❤️.txt - rename the file - verify that the first 4 characters are correctly selected, which didn't work before the patch. M +8 -9 src/kitemviews/kfileitemlistwidget.cpp M +1 -1 src/kitemviews/kfileitemlistwidget.h M +12 -1 src/kitemviews/kstandarditemlistwidget.cpp M +3 -1 src/kitemviews/kstandarditemlistwidget.h https://invent.kde.org/system/dolphin/-/commit/021365dceb590a14bfcdb904ea05ffdd69b7d663 Git commit 34d3f770bcc5759f46b0ccc8fcad9956c7fbed0a by Yifan Zhu. Committed on 13/08/2024 at 16:29. Pushed by meven into branch 'release/24.08'. KStandardItemListWidget: select by number of unicode chars Previously during rename, the number of QChar is used for selection, which might be different from number of unicode characters. Test plan: - create the file zz❤️❤️.txt - rename the file - verify that the first 4 characters are correctly selected, which didn't work before the patch. (cherry picked from commit 021365dceb590a14bfcdb904ea05ffdd69b7d663) Co-authored-by: Yifan Zhu <fanzhuyifan@gmail.com> M +8 -9 src/kitemviews/kfileitemlistwidget.cpp M +1 -1 src/kitemviews/kfileitemlistwidget.h M +12 -1 src/kitemviews/kstandarditemlistwidget.cpp M +3 -1 src/kitemviews/kstandarditemlistwidget.h https://invent.kde.org/system/dolphin/-/commit/34d3f770bcc5759f46b0ccc8fcad9956c7fbed0a |