-
-
Notifications
You must be signed in to change notification settings - Fork 214
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
Web Apps: switched combobox to select2 #28701
Merged
Merged
Changes from 14 commits
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
ab38860
Fixed bindings for dropdown template
orangejenny 288c96f
Switched combobox to use dropdown template and select2 for rendering
orangejenny aaacd10
Added translations
orangejenny 439b922
Combined DropdownEntry and ComboboxEntry to use the same UI
orangejenny 11575ac
Removed dropdown help text for consistency with prod
orangejenny 6ea3bcb
Removed validation
orangejenny 561a4cb
Added select2 to app preview template
orangejenny 70f0448
Updated to handle blank default value
orangejenny 85f8ded
Extracted ComboboxEntry.filter again to make it available to tests
orangejenny d7783e3
Fixed lint error
orangejenny ac807f5
Added select2 to tests
orangejenny 36ac455
Restored onPreProcess and validation code
orangejenny 367dd0d
Updated tests to reflect changes in entry's options
orangejenny 0c19009
Restored additional validation code
orangejenny 6c72078
Restored valueAllowUnset binding
orangejenny 3fb2a76
Updated ComboboxEntry to use index for both answer and rawAnswer
orangejenny 1482c36
Updated select2 so UI reflects programmatic value
orangejenny c39f408
Updated dropdown and combobox to clear answer when options change
orangejenny 242a564
Moved clearing logic to dropdown instead of combobox
orangejenny File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
8 changes: 8 additions & 0 deletions
8
corehq/apps/cloudcare/templates/cloudcare/spec/form_entry/mocha.html
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i'm guessing
pureComputed
doesn't have thesubscribe
option?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It does, but the subscribers shouldn't have side effects. I think that rendering the select2 counts as a side effect, although I'm totally certain. I'm looking at the "When not to use a pure computed observable" section in these docs: https://knockoutjs.com/documentation/computed-pure.html