Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix alt+arrow navigation for dynamics #26848

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mike-spa
Copy link
Contributor

Resolves: #26249 (part of)
Did my best

@mike-spa mike-spa requested a review from avvvvve February 28, 2025 18:36
@MarcSabatella
Copy link
Contributor

This seems to work well from my perspective, thanks!

One thing I just realized, though - not new to the PR, but I don't actually see a way to put the dynamic into Edit mode via keyboard, should I wish to change the text. I can tab into it to change the dynamic that way, but F2 toggles ther popup rather than placing the dynamic into Edit mode. I can get there via the context menu, and Ctrl+D puts me in edit mode right away, so this is hardly a showstopper. But F2 did work to enable edit mode in previous releases.

I suppose it's possible there is a reason for F2 toggling the pop-up, but if not, I wonder if a similar "one line fix" would bypass that and allow it to engage edit mode like it does for other text?

@avvvvve
Copy link

avvvvve commented Feb 28, 2025

@MarcSabatella Good catch! Logged that separately here: #26850


@mike-spa this looks good to me! approved

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.

Keyboard interaction issues with dynamics popup
3 participants