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

Refactor autocomplete #772

Merged
merged 1 commit into from
Nov 23, 2023
Merged

Refactor autocomplete #772

merged 1 commit into from
Nov 23, 2023

Conversation

jochenklar
Copy link
Member

Description

This PR fixes some behavior for the autocomplete widget and adds a new freeautocomplete widget which can also store arbitrary inputs. The values are then using text and not option to store the input, just like regular text inputs.

Related issue: #600

@jochenklar jochenklar self-assigned this Oct 6, 2023
@jochenklar jochenklar added this to the 2.1.0 milestone Oct 6, 2023
@MyPyDavid
Copy link
Member

I've add a question with the the freeautocomplete widget to the test fixture catalog.
When the tests pass, it should be fine right?

@MyPyDavid
Copy link
Member

no, that breaks all the import tests and requires a lot of refactoring of tests and fixtures I think..

Copy link
Member

@MyPyDavid MyPyDavid left a comment

Choose a reason for hiding this comment

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

could use some more testing...

@jochenklar jochenklar merged commit 9578643 into dev-2.1.0 Nov 23, 2023
29 checks passed
@jochenklar jochenklar deleted the refactor_autocomplete branch December 12, 2023 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants