Skip to content

Conversation

@MarvinKlein1508
Copy link
Collaborator

@MarvinKlein1508 MarvinKlein1508 commented Oct 10, 2025

Pull Request

📖 Description

This PR fixes the FocusAsync methods for the FluentAutoComplete component. It overwrites both FocusAsync methods and forwards it to the new instance of the Element property.

🎫 Issues

Fix: #4228

👩‍💻 Reviewer Notes

There is still an a11y bug in one special case. This is broken outside of this PR as well so I'm gonna open another issue for this. See: #4231

📑 Test Plan

✅ Checklist

General

  • I have added tests for my changes.
  • I have tested my changes.
  • I have updated the project documentation to reflect my changes.
  • I have read the CONTRIBUTING documentation and followed the standards for this project.

Component-specific

  • I have added a new component
  • I have added Unit Tests for my new component
  • I have modified an existing component
  • I have validated the Unit Tests for an existing component

⏭ Next Steps

@vnbaaij vnbaaij modified the milestones: v4.13.0, v4.13.1 Oct 10, 2025
@vnbaaij vnbaaij merged commit cbae1fb into microsoft:dev Oct 10, 2025
2 checks passed
@vnbaaij vnbaaij changed the title [AutoComplete] Overwrite FocusAsync in FluentAutoComplete [AutoComplete] Override FocusAsync in FluentAutoComplete Oct 10, 2025
vnbaaij added a commit that referenced this pull request Oct 10, 2025
Co-authored-by: Vincent Baaij <vnbaaij@outlook.com>
@vnbaaij vnbaaij modified the milestones: v4.13.1, v4.13.0 Oct 10, 2025
@vnbaaij
Copy link
Collaborator

vnbaaij commented Oct 10, 2025

I cherry picked this PR into main so it can be included in the v4.13.0 release

@MarvinKlein1508 MarvinKlein1508 deleted the fix-4228 branch October 10, 2025 10:50
vnbaaij added a commit that referenced this pull request Oct 30, 2025
* Update NuGet package for fixing CVE. See dotnet/announcements#327

* Non-altering change to force mirroring

* Clean up issue tester

* Small text formatting change to force deploy

* Update workflows

* Update README

* Update to latest JavaScript SDK

* Add global.json

* Update workflow to use .NET 9

* Remove net10.0 TFM for now

* Set dotnet 9.0.204

* Update deploy_demo.yml to use 9.0.204

* Update home page

* Also *temporarily* remove .NET 10 TFM for DataGrid Adapters

* Update workflows

* Try to get preview docs up again

* Use latest .NET 9 SDK for GH Actions

* Remove mention of v3 site

* Update global.json

* Test deploy to new SWA

* Update JavaScript SDK

* Try older JavaScript SDK

* Use .204 SDK on all actions

* Newer JavaScript SDK with .NET .204 SDK

* Deploy demo to 2 SWAs

* Fix id's

* Evrything tied to .204 and add sub subscription

* Get sources copied

* Try again

* Tweak message

* Try again

* Try again

* Try swa.config.json

* Try extension

* Revert config.json change

* Use correct codeql action

* Include build step in actions

* Set SDK for ADO pipelines

* Add es metadata (#4131)

* Merge pull request 53130 from invBootstrap into main

* Update TSAOptions codebaseName (#4196)

* Update TSAOptions codebaseName

* Try to start the Mirroring

* Update pipeline for real signing with PME enforcement

* Add a trigger on main and a tag (v*)

* Overwrite FocusAsync in FluentAutoComplete (#4230)

Co-authored-by: Vincent Baaij <vnbaaij@outlook.com>

* Update whats new after cherry pick

* Try to fix deploy error

* Different fix

* Take 3

* Take 4

* Take 5

* Take 6

* Disable main brancg in the All-lib YAML file

* Prep for next release

* Use correct SDK version, update comment

---------

Co-authored-by: Denis Voituron <dvoituron@outlook.com>
Co-authored-by: MerlinBot <MerlinBot>
Co-authored-by: Marvin Klein <32510006+MarvinKlein1508@users.noreply.github.com>
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.

fix: FocusAsync() method throws exception in FluentAutocomplete

2 participants