Bug 444680 - Right clicking on header shows file context-menu instead of column role context-menu
Summary: Right clicking on header shows file context-menu instead of column role conte...
Status: RESOLVED FIXED
Alias: None
Product: dolphin
Classification: Applications
Component: view-engine: details mode (show other bugs)
Version: 21.08.2
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Tom Lin
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-10-30 19:52 UTC by Tom Lin
Modified: 2022-01-16 14:01 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 22.04
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Tom Lin 2021-10-30 19:52:41 UTC
SUMMARY

In details view, if a highlighted item is directly behind (obstructed by) the header, right-clicking in the header within the obstructed highlight rect will trigger the item's context-menu and not the header's.

STEPS TO REPRODUCE
1. Toggle details view and navigate to some location that allows scrolling.
2. Highlight any item and scroll the panel such that the highlighted item gets covered by the header.
3. Right click on the header at the covered item's *highlighted* rectangular region. Note that this only occurs in the highlight rect, and not anywhere in the row.

OBSERVED RESULT

The context-menu of the selected item gets triggered.

EXPECTED RESULT

The header's role configuration menu (the menu to show/hide columns, etc.) should be triggered.

This is reproducible in both 21.08.2 and 21.08.0, I have not tested this on any other version yet.

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION
Comment 1 Tom Lin 2021-10-30 19:59:03 UTC
Testing a bit more, it's not necessary to highlight anything.
The bug occurs if you simply right-click on the header in a location that needs scrolling. Essentially, file item's text (and not the entire row) rect hit check was done before the header's so the click events gets swallowed.
Comment 2 Tom Lin 2021-10-30 20:01:26 UTC
This issue is most obvious if https://invent.kde.org/system/dolphin/-/merge_requests/278 gets merged, I've added a fix in that MR for now.
Comment 3 Felix Ernst 2022-01-16 14:01:45 UTC
Git commit d3839617193e92463806580699caa595c892b8a6 by Felix Ernst, on behalf of Tom Lin.
Committed on 16/01/2022 at 14:01.
Pushed by felixernst into branch 'master'.

Full row highlight implementation

This commit implements full-row selection and hover highlights for the
details view mode.

This commit also contains fixes for 444680, 444753, both uncovered
during this change.
Related: bug 181438, bug 444753
FIXED-IN: 22.04

M  +143  -36   src/kitemviews/kitemlistcontroller.cpp
M  +16   -0    src/kitemviews/kitemlistheader.cpp
M  +9    -0    src/kitemviews/kitemlistheader.h
M  +51   -5    src/kitemviews/kitemlistview.cpp
M  +11   -0    src/kitemviews/kitemlistview.h
M  +32   -0    src/kitemviews/kitemlistwidget.cpp
M  +10   -1    src/kitemviews/kitemlistwidget.h
M  +4    -0    src/kitemviews/kstandarditemlistview.cpp
M  +42   -10   src/kitemviews/kstandarditemlistwidget.cpp
M  +6    -0    src/kitemviews/kstandarditemlistwidget.h
M  +92   -34   src/kitemviews/private/kitemlistheaderwidget.cpp
M  +15   -0    src/kitemviews/private/kitemlistheaderwidget.h
M  +4    -0    src/settings/dolphin_detailsmodesettings.kcfg
M  +1    -0    src/views/dolphinitemlistview.cpp
M  +17   -0    src/views/dolphinview.cpp
M  +1    -0    src/views/dolphinview.h

https://invent.kde.org/system/dolphin/commit/d3839617193e92463806580699caa595c892b8a6