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/imagechoice-component #1091
base: dev
Are you sure you want to change the base?
Conversation
imagechoice-component-Updated dialog option label and corresponding java files
imagechoice-component-Updated imagechoiceview js file
imagechoice-component-update editDialog js
Updated imagechoice.html file
imagechoice-component- Updated ImageChoiceImpl java file
Feature/imagechoice component
Feature/imagechoice dev
imagechoice-component - updated imagechoiceview js and imagechoiceimp…
imagechoice-component-updated imagechoiceview js file
imagechoice-component - Added Unit Test cases
Feature/imagechoice dev
imagechoice-component- updated readme file
imagechoice-component - removed notnull
imagechoice-component - Added ui.tests - authoring spec.js
IMAGE("image"); | ||
IMAGE("image"), | ||
|
||
IMAGECHOICE("imagechoice"); |
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.
fieldType 'imagechoice' does not exists, I believe it should be 'checkbox'
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.
The reason for creating a new field type is that the image choice component is a composite multi/single selection type, and it will not store values as an enum.
Feature/imagechoice dev
imagechoice component - updated ui test runtime js and added sample
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.
Remove the code at line 130(this.toggle(enabled, FormView.Constants.ARIA_DISABLED, true);), as it will add aria-disabled attribute on the parent div. Aria-disabled attribute is applied on the input, not on the div.
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.
Removed the code at line number - 130.
ImageChoiceComponent - Implemented Code Review feedback [Remove the code at line 130(this.toggle(enabled, FormView.Constants.ARIA_DISABLED, true);), as it will add aria-disabled attribute on the parent div. Aria-disabled attribute is applied on the input, not on the div.]
Adaptive Form Image Choice (v1)
Description
This PR with code changes for Adaptive Form Image Choice Component (V1)
This component still needs to be updated and tested accordingly.
The code changes made so far reflect the current progress.
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: