Skip to content

Calendar: avoid opening menu on unselected component#2028

Merged
Knicksaas merged 1 commit intoreleases/26.2from
features/knicksaas/26.2/451116_avoid_invalid_calendar_menu
Feb 20, 2026
Merged

Calendar: avoid opening menu on unselected component#2028
Knicksaas merged 1 commit intoreleases/26.2from
features/knicksaas/26.2/451116_avoid_invalid_calendar_menu

Conversation

@Knicksaas
Copy link
Member

The javascript 'contextmenu' event is not consistent with its order. On some machines, it is triggered right after right-click 'mousedown', on other machines after 'mouseup'. This can lead to an unexpected behavior, when the right-click is started outside a component and is finished on a component. Because the selection is applied on 'mousedown', no component is selected, but the context menu of the component is opened. This change cancels the opening of the popup window When this case occurs.

451116

@Knicksaas Knicksaas self-assigned this Feb 18, 2026
The javascript 'contextmenu' event is not consistent with its order. On
some machines, it is triggered right after right-click 'mousedown', on
other machines after 'mouseup'. This can lead to an unexpected
behavior, when the right-click is started outside a component and is
finished on a component. Because the selection is applied on
'mousedown', no component is selected, but the context menu of the
component is opened. This change cancels the opening of the popup window
when this case occurs.

451116
@Knicksaas Knicksaas force-pushed the features/knicksaas/26.2/451116_avoid_invalid_calendar_menu branch from 6aa2ac8 to 834c8b7 Compare February 18, 2026 14:05
@Knicksaas Knicksaas requested a review from jwinkens February 18, 2026 14:33
Copy link
Member

@jwinkens jwinkens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review ok

@Knicksaas Knicksaas merged commit c5b71a0 into releases/26.2 Feb 20, 2026
4 checks passed
@Knicksaas Knicksaas deleted the features/knicksaas/26.2/451116_avoid_invalid_calendar_menu branch February 20, 2026 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants