Skip to content

Modify focus events for TextBox B#509

Open
WillEhrendreich wants to merge 1 commit into
fsprojects:masterfrom
WillEhrendreich:patch-1
Open

Modify focus events for TextBox B#509
WillEhrendreich wants to merge 1 commit into
fsprojects:masterfrom
WillEhrendreich:patch-1

Conversation

@WillEhrendreich
Copy link
Copy Markdown

Updated focus handling for TextBox B to set focus state. it's a 2 character documentation change that doesn't have any tests associated with it, most likely, won't affect builds, etc. I don't see an advantage of going through the ceremony of the checklist items, as if it were a real code change.

Updated focus handling for TextBox B to set focus state.
Copilot AI review requested due to automatic review settings May 6, 2026 17:43
@WillEhrendreich WillEhrendreich requested a review from JaggerJo as a code owner May 6, 2026 17:43
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the “Managing Focus” documentation snippet to correctly track focus state for TextBox B (previously, TextBox B focus events were updating the TextBox A focus state).

Changes:

  • Fix TextBox B’s onGotFocus handler to set textBoxBFocus
  • Fix TextBox B’s onLostFocus handler to set textBoxBFocus

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

3 participants