-
Notifications
You must be signed in to change notification settings - Fork 540
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
Added Placeholder to ChoiceSet #4811
Conversation
There was an error handling pipeline event 820873f4-824a-4ff9-a38f-0a2886cb9a58. |
…aptiveCards into jwoo/choice-set-placeholder
@@ -20,10 +20,14 @@ | |||
"type": "string", | |||
"description": "The initial choice (or set of choices) that should be selected. For multi-select, specify a comma-separated string of values." | |||
}, | |||
"placeholder": { | |||
"type": "string", | |||
"description": "Description of the input desired. Displayed when no selection has been made, style is compact and multi selection is false." |
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.
Thanks for adding this Joseph! Can you tweak the format a bit, something like: "Description of the input desired. Only visible when no selection has been made, the style
is compact
and isMultiSelect
is false
." #Resolved
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.
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.
You're missing changes in the .idl file, that's why the UWP build is failing
thanks In reply to: 496028907 [](ancestors = 496028907) |
…aptiveCards into jwoo/choice-set-placeholder
Could we get a simple sample card that uses this? If we put it in the Elements directory and name it Input.ChoiceSet.Placeholder.json it can serve as a website sample as well. #Resolved Refers to: schemas/src/elements/inputs/Input.ChoiceSet.json:1 in 90f7904. [](commit_id = 90f7904, deletion_comment = False) |
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.
Please add the Input.ChoiceSet.Placeholder sample to the v1.3 folder
…aptiveCards into jwoo/choice-set-placeholder
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.
* Fixed the crash issue * Fixed Bleeding Crash Issue * finishing up changes for bleed update * work in progress * updated ChoiceSetInput.cpp * ios change * schema & uwp changes * Updated specs * Android Changes * addressed review comments * Updated specs * Added placeholder text example to test cards * added a new card * added UWP test images * Fixed bad merge * updated schema * Updated specs Co-authored-by: Adaptive Cards Specs CI <donotreply@microsoft.com> Co-authored-by: almedina-ms <35784165+almedina-ms@users.noreply.github.com> Co-authored-by: shalinijoshi19 <shalinij@microsoft.com>
Related Issue
reference #3885
Fixes #4810
Description
Added placeholder text for Input.ChoiceSet when it is single select and compact style.
How Verified
How you verified the fix, including one or all of the following:
Microsoft Reviewers: Open in CodeFlow