dail8859/NotepadNext

Editor Context Menu Location

Closed

#780 opened on May 27, 2025

View on GitHub
 (1 comment) (0 reactions) (0 assignees)C++ (905 forks)batch import
good first issue

Repository metrics

Stars
 (14,472 stars)
PR merge metrics
 (Avg merge 9d 1h) (3 merged PRs in 30d)

Description

Description

When right-clicking in the editor there is inconsistent behavior because some commands use the position of the mouse to determine if some things can be used (e.g. a URL), but others need the current caret position (markers).

Describe the solution you'd like

It seems most editors rely on the logic:

  • If the right click is inside a selection, do not change the selection at all, then open the context menu based on the current selection.
  • If the right click is outside the selection, place the caret as the mouse position, then open the context menu based on the new location.

Describe alternatives you've considered

None - Most editors seem to have the described behavior.

Contributor guide

Research direction
The issue asks to change context menu behavior to match common editors. Focus on the editor's context menu event handler and selection/caret logic. Look for existing code that handles right click events and modifies the caret position. Typically, such code is in the editor widget class. The desired behavior is: if right click is within selection, keep selection and base menu on selection; otherwise, move caret to click position and base menu on new caret location. Check the repository's current implementation to identify where changes are needed.
Tech stack
cpp
Domain
frontend
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
C++Qt framework
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60