-
Notifications
You must be signed in to change notification settings - Fork 51
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
Feature: Autocomplete/dropdown or any other assisted input with generic conditionals #42
Comments
I started tackling this last night and made some decent progress. I'm going to try and have it ready for some testing by next week. |
Cool, assigned this to you and the v0.3 milestone |
@jeremyfelt still want to try to swing this for v0.4? |
This got pretty ugly pretty quick. I'm guessing it's a no go for v0.4, but deserves future thought. I'll take a peek soon. |
Brainstorming here... Ignoring quick edit for the moment, just focusing on new ad code.
So...
See, ugly. :) Thoughts @danielbachhuber @rinatkhaziev? |
Start autocompleting at 3 chars, limit to 100 items. This way we won't do any expensive queries.
Makes sense
See the first bullet point
Filter would definitely be nice. Not that ugly :) |
Good point. With Chosen, the user could just finish typing if it doesn't match one of the 100. Much less ugly. |
@jeremyfelt will you be able to finish this up in the next day or so? |
I'm going to try and toss some time at it this evening or tomorrow, but the outlook is unlikely at finishing it up for 0.4. |
Let's just put it in v0.5 then, and we can keep the momentum going by releasing in a month or so. Please do the work on a feature branch when you do work on it. |
Awesome, sounds good. |
No branch apparently existing in GitHub for this work. |
Per @jeremyfelt:
it would be cool if we could modify the input fields for each conditional either through filters or just by default. For example – If is_category() is selected as the conditional, a drop down list of existing categories would be more pleasant than a text field.
The text was updated successfully, but these errors were encountered: