Skip to content

Obsolete VisualElement.FocusChangeRequested event#16750

Merged
PureWeen merged 1 commit intomainfrom
fix-14299
Aug 15, 2023
Merged

Obsolete VisualElement.FocusChangeRequested event#16750
PureWeen merged 1 commit intomainfrom
fix-14299

Conversation

@jfversluis
Copy link
Copy Markdown
Member

Description of Change

As indicated by a todo in code, obsoletes the VisualElement.FocusChangeRequested event. This is now handled through the VisualElement mapper with MapFocus by setting the IView.Focus property.

Issues Fixed

Fixes #14299

@jfversluis jfversluis added t/breaking 💥 legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor labels Aug 15, 2023
@jfversluis jfversluis requested a review from a team as a code owner August 15, 2023 08:34
@ghost
Copy link
Copy Markdown

ghost commented Aug 15, 2023

🚨 API change(s) detected @davidbritch FYI

@PureWeen PureWeen enabled auto-merge (squash) August 15, 2023 10:14
@PureWeen PureWeen disabled auto-merge August 15, 2023 10:14
@PureWeen PureWeen enabled auto-merge (squash) August 15, 2023 10:14
@PureWeen PureWeen merged commit 48e5caa into main Aug 15, 2023
@PureWeen PureWeen deleted the fix-14299 branch August 15, 2023 12:15
@github-actions github-actions bot locked and limited conversation to collaborators Dec 7, 2023
@Eilon Eilon added area-controls-general General issues that span multiple controls, or common base classes such as View or Element and removed legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor labels May 13, 2024
@samhouts samhouts added the fixed-in-8.0.0-rc.1.9171 Look for this fix in 8.0.0-rc.1.9171 label Aug 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-controls-general General issues that span multiple controls, or common base classes such as View or Element fixed-in-8.0.0-rc.1.9171 Look for this fix in 8.0.0-rc.1.9171 t/breaking 💥 t/housekeeping ♻︎

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Obsolete FocusChangeRequested

4 participants