Bug 375548 - context menu not opened every other click (Mac)
Summary: context menu not opened every other click (Mac)
Status: RESOLVED WORKSFORME
Alias: None
Product: frameworks-ktexteditor
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: 5.29.0
Platform: Compiled Sources macOS
: NOR normal
Target Milestone: ---
Assignee: KWrite Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-01-25 15:07 UTC by RJVB
Modified: 2022-02-07 04:37 UTC (History)
1 user (show)

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


Attachments
side-ways related: possible optimisation for KTextEditor::ViewPrivate::contextMenu() (1017 bytes, text/plain)
2017-01-25 15:13 UTC, RJVB
Details

Note You need to log in before you can comment on or make changes to this bug.
Description RJVB 2017-01-25 15:07:27 UTC
For a while now I've been seeing an issue in KDevelop on Mac where the context menu doesn't open every other right-click:

- a right-click in a newly opened or switched-to document opens the context menu
- dismiss the menu (e.g. with Escape), right-click again; menu doesn't open
- right-click once more; menu opens.

The context menu does open reliably in other views (and on Linux).

I don't have this issue in Kate but I do notice that `KTextEditor::ViewPrivate::aboutToShowContextMenu()` is not called when I click and the context menu doesn't appear.

That may make it a Qt issue (indeed I've been noticing in certain dialogs that I now need 2 left-clicks to activate the "next" button where before (and on Linux) I only need to click once); it's quite possible the issue started when I updated to Qt 5.7.

If not a Qt issue, can it be the result of a change in the way Qt signals are being handled in KTextEditor?
Comment 1 RJVB 2017-01-25 15:13:30 UTC
Created attachment 103634 [details]
side-ways related: possible optimisation for KTextEditor::ViewPrivate::contextMenu()

This is a change I tested to see if the repeated reconnecting (disconnect/connect) in `KTextEditor::ViewPrivate::contextMenu()` caused clicks to get lost. Apparently it doesn't, but maybe it's a good idea to make this change anyway, despite the fact it naughtily violates the methods "const" qualifier?
Comment 2 Christoph Cullmann 2022-01-08 21:32:09 UTC
Hi, is that still an issue with the current version?
For the optimization: no idea if that is good idea, if in doubt, would not touch that code.
Comment 3 RJVB 2022-01-08 23:18:26 UTC
Sorry, no idea, I'm stuck at 5.60.0 for now. That version does still have issues with the context menu in KTextEditor views, especially in KDevelop.
Comment 4 Bug Janitor Service 2022-01-23 04:37:29 UTC
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least
15 days. Please provide the requested information as soon as
possible and set the bug status as REPORTED. Due to regular bug
tracker maintenance, if the bug is still in NEEDSINFO status with
no change in 30 days the bug will be closed as RESOLVED > WORKSFORME
due to lack of needed information.

For more information about our bug triaging procedures please read the
wiki located here:
https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please
mark the bug as REPORTED so that the KDE team knows that the bug is
ready to be confirmed.

Thank you for helping us make KDE software even better for everyone!
Comment 5 Bug Janitor Service 2022-02-07 04:37:12 UTC
This bug has been in NEEDSINFO status with no change for at least
30 days. The bug is now closed as RESOLVED > WORKSFORME
due to lack of needed information.

For more information about our bug triaging procedures please read the
wiki located here:
https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

Thank you for helping us make KDE software even better for everyone!