fix: Simplify autocomplete input for V3

[TomasEng]

Description

Replaced the old EditAutocomplete component in the V3 forms editor with a simpler one. This was necessary to remove the use of asChild in our code. The component was also overly complex; a simple NativeSelect should be enough for this field.

Master

Spiller.inn.2024-12-11.134954.mp4

This branch

Spiller.inn.2024-12-11.123718.mp4

Note

Actually, the autocomplete property should support multiple values, so a multiselect combobox would be the most appropriate solution here. But this was not supported in the main branch, so we should create a separate issue to fix that. We also need to check if it is supported by Apps.

Related Issue(s)

• Remove asChild support from the component library #14006

Verification

• Your code builds clean without any errors or warnings
• Manual testing done (required)
• Relevant automated test added (if you find this hard, leave it and we'll help out)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.52%. Comparing base (c6690b7) to head (612e2ba).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #14262   +/-   ##
=======================================
  Coverage   95.51%   95.52%           
=======================================
  Files        1826     1829    +3     
  Lines       23786    23775   -11     
  Branches     2757     2754    -3     
=======================================
- Hits        22720    22710   -10     
+ Misses        808      807    -1     
  Partials      258      258           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[standeren]

🥳

[standeren]

Tested this locally now and it looks as expected and behaves as expected until I try to unset the property.

I see that when I remove the autocomplete property from a component the autocomplete property is set to an empty string. It is not a huge problem (I suppose), but we should not create unused config in the apps.

[TomasEng]

You're right, @standeren. I'll fix the update function so that the property is removed when it receives an empty string.

[standeren]

Tested again now. Everything works as expected. Added the skip-manual-testing label so we can merge it 🫡