Summary: | Breadcrumb drop-down items are sometimes accidentally activated | ||
---|---|---|---|
Product: | [Frameworks and Libraries] frameworks-kio | Reporter: | Алексей Шилин <rootlexx> |
Component: | general | Assignee: | David Faure <faure> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | bugseforuns, kdelibs-bugs, nate |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | https://commits.kde.org/kio/4a6a3b81bedca043bbe2c620cea296f9f69e5eea | Version Fixed In: | 5.44 |
Description
Алексей Шилин
2017-05-28 20:25:51 UTC
Can confirm the issue with KDE Frameworks 5.43. Didn't you submit a patch for this? If so, can you drop the link in a comment here? Shame Bugzilla and Phabricator aren't linked up in that way.. (In reply to Nate Graham from comment #1) > Didn't you submit a patch for this? I did. You've even tested it. :) I originally (incorrectly) reported the bug against Dolphin, and now I just reassigned it to KIO so that FIXED-IN tag works. No wonder this bug report it seemed so familiar! :) Git commit 4a6a3b81bedca043bbe2c620cea296f9f69e5eea by Алексей Шилин. Committed on 23/02/2018 at 17:42. Pushed by aleksejshilin into branch 'master'. Fix unintentional breadcrumb menu item activation Summary: Since breadcrumb menu is opened on mouse press, it may receive the corresponding mouse release event which may unintentionally activate the random item which happens to be under mouse pointer. This commit makes the menu ignore the first mouse release event unless the pointer was moved. It fixes the issue and still allows 'Press mouse button' -> 'Move to an item' -> 'Release the button' usage scenario. FIXED-IN: 5.44 Reviewers: #frameworks, dfaure Reviewed By: dfaure Subscribers: broulik, ngraham Tags: #frameworks Differential Revision: https://phabricator.kde.org/D8958 M +30 -6 src/filewidgets/kurlnavigatormenu.cpp M +5 -0 src/filewidgets/kurlnavigatormenu_p.h https://commits.kde.org/kio/4a6a3b81bedca043bbe2c620cea296f9f69e5eea |