Delete old samples that don't have latest features (causing accessibility issues) #7106
Conversation
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
PankajBhojwani
changed the title
PankajBhojwani
commented
Mar 9, 2022
PankajBhojwani
commented
Mar 9, 2022
carlos-zamora
requested changes
Mar 9, 2022
There was a problem hiding this comment.
Does the
versionnumber at the top of the file need to be updated? Thelabelattribute seems to be working fine without updating the version number but I'm not sure if we should still update it for consistency?
Just blocking over this comment. We definitely should update the version number because an author may copy-and-paste this card directly and expect it to work in 1.0. For reference, unsupported features are generally just dropped. So if you took this new card and put it in a v1.0 renderer, the label should just be dropped entirely (wow, so this card would look really bad haha).
A question I have though is, should we move this card out of the v1.0 folder then? Would that be bad for testing? Need insight from @paulcam206 or @licanhua here.
ghost
added
the
I think in this case it's probably fine to leave it (or update if you feel like pushing another commit) -- labels shipped long enough ago that at this point all major partners support them. As for the various |
paulcam206
approved these changes
Mar 9, 2022
paulcam206
requested changes
Mar 9, 2022
That's a good call! Deleted the files |
PankajBhojwani
changed the title
carlos-zamora
approved these changes
Mar 10, 2022
paulcam206
approved these changes
Mar 10, 2022
JeanRoca
pushed a commit
to JeanRoca/AdaptiveCards
that referenced
this pull request
Apr 5, 2022
…lity issues) (microsoft#7106) * updates to samples * phone number regex * delete files instead Co-authored-by: Pankaj Bhojwani <[email protected]> Co-authored-by: Vsevolod <[email protected]>
paulcam206
pushed a commit
that referenced
this pull request
Apr 14, 2022
…lity issues) (#7106) * updates to samples * phone number regex * delete files instead Co-authored-by: Pankaj Bhojwani <[email protected]> Co-authored-by: Vsevolod <[email protected]>
licanhua
added a commit
that referenced
this pull request
Sep 14, 2022
* Updating Bot Framework links (#6686) Co-authored-by: RahulAmlekar <[email protected]> * Scenario cards + Website indentation fix (#6733) * Changed teams target version to 1.4 * added RTL scenario card * Added scenario card for tooltip and isEnabled * Added order confirmation scenario * Added flight update table scenario * Added restaurant order scenario card * Added app login scenario * Fixed indentation in schema explorer * Testing application login on the samples page * Added extended 1.5 scenario cards * Fixed boolean and duplicate id on cards * added flightupdatetable and restaurant order to ignored cards for .NET tests Co-authored-by: Canhua Li <[email protected]> * Removing "Preview" from Viva Connections (#6791) Co-authored-by: Paul Campbell <[email protected]> * Added blog posts for oct and nov (#6873) * Changed teams target version to 1.4 * Added october and november blog posts * fixed emojis * testing font weights for emoji * removed bold from emojis as not required to render Co-authored-by: Canhua Li <[email protected]> Co-authored-by: almedina-ms <[email protected]> * make columnsets clickable (#7042) Co-authored-by: Canhua Li <[email protected]> * fix merge conflicts with berlin containers in adaptivecards-designer.ts * Convey Ctrl+M shortcut key to move out of code editor (#7072) * provide hint about ctrl+m toggling tab behavior * rephrase Co-authored-by: Pankaj Bhojwani <[email protected]> Co-authored-by: Canhua Li <[email protected]> * fixed same merge conflict in another commit * color fix (#7081) Co-authored-by: Pankaj Bhojwani <[email protected]> * update headers (#7082) * Add title to iframe component in December 2019 blog post (#7083) * add title * spacing Co-authored-by: Pankaj Bhojwani <[email protected]> * change text color (#7091) Co-authored-by: Pankaj Bhojwani <[email protected]> * add aria label (#7090) Co-authored-by: Pankaj Bhojwani <[email protected]> * update tab indices accordingly (#7119) Co-authored-by: Vsevolod <[email protected]> * Adjust contrast on designer peer buttons (#7110) * Update role of 'Select Host App' combobox (#7118) Co-authored-by: root <[email protected]> Co-authored-by: Vsevolod <[email protected]> * Delete old samples that don't have latest features (causing accessibility issues) (#7106) * updates to samples * phone number regex * delete files instead Co-authored-by: Pankaj Bhojwani <[email protected]> Co-authored-by: Vsevolod <[email protected]> * removed carousel samples * fixed merge conflict line 8 of package.json * fix titles (#7130) Co-authored-by: Pankaj Bhojwani <[email protected]> * fixed merge conflict in ADCIOSVisualizerUITests * add tooltip on initialization (#7126) * more titles to iframes (#7135) Co-authored-by: Pankaj Bhojwani <[email protected]> * add alt text (#7131) Co-authored-by: Pankaj Bhojwani <[email protected]> * use active border instead of white (#7122) * add landmark (#7145) * Use HTML lists for hyperlinks (#7127) * Use HTML lists for hyperlinks * consistent spacing in css file Co-authored-by: root <[email protected]> Co-authored-by: Vsevolod <[email protected]> Co-authored-by: PankajBhojwani <[email protected]> * Add a caption to the table present in the 'help' dialog (#7149) * move keyboard shortcut text to caption * use innerText instead * don't let focus fall on card designer surface (#7153) * add 1 px left margin (#7142) * Add underline styling to hyperlinks (#7157) Co-authored-by: root <[email protected]> Co-authored-by: Vsevolod <[email protected]> * Allow footer hyperlinks to wrap (#7147) Co-authored-by: PankajBhojwani <[email protected]> * Website updates (#7163) * Changed teams target version to 1.4 * Added october and november blog posts * fixed emojis * testing font weights for emoji * removed bold from emojis as not required to render * added blogs posts * fixed video titles * updated roadmap links * removed repeated sentence Co-authored-by: Canhua Li <[email protected]> * [JS] Remove `role='menubar'` and `role='menuitem'` from `ActionCollection` (#6763) Fixes #6425 Fixes #6019 Related #4859 * sync loc file * more package-locks * cherry pick 646503b * return focus after closing (#7155) * final cherry picks for 5e0f39f and 67187d0 * update designer * revert package-lock.json changes Co-authored-by: Rahul Amlekar <[email protected]> Co-authored-by: RahulAmlekar <[email protected]> Co-authored-by: J.P. Roca <[email protected]> Co-authored-by: Siddharth Gulati <[email protected]> Co-authored-by: Paul Campbell <[email protected]> Co-authored-by: almedina-ms <[email protected]> Co-authored-by: Roy Nehoran <[email protected]> Co-authored-by: naramaka <[email protected]> Co-authored-by: PankajBhojwani <[email protected]> Co-authored-by: Pankaj Bhojwani <[email protected]> Co-authored-by: Will Shown <[email protected]> Co-authored-by: Vsevolod <[email protected]> Co-authored-by: Carlos Zamora <[email protected]> Co-authored-by: root <[email protected]>
licanhua
added a commit
that referenced
this pull request
Sep 15, 2022
* Updating Bot Framework links (#6686) Co-authored-by: RahulAmlekar <[email protected]> * Scenario cards + Website indentation fix (#6733) * Changed teams target version to 1.4 * added RTL scenario card * Added scenario card for tooltip and isEnabled * Added order confirmation scenario * Added flight update table scenario * Added restaurant order scenario card * Added app login scenario * Fixed indentation in schema explorer * Testing application login on the samples page * Added extended 1.5 scenario cards * Fixed boolean and duplicate id on cards * added flightupdatetable and restaurant order to ignored cards for .NET tests Co-authored-by: Canhua Li <[email protected]> * Removing "Preview" from Viva Connections (#6791) Co-authored-by: Paul Campbell <[email protected]> * Added blog posts for oct and nov (#6873) * Changed teams target version to 1.4 * Added october and november blog posts * fixed emojis * testing font weights for emoji * removed bold from emojis as not required to render Co-authored-by: Canhua Li <[email protected]> Co-authored-by: almedina-ms <[email protected]> * make columnsets clickable (#7042) Co-authored-by: Canhua Li <[email protected]> * fix merge conflicts with berlin containers in adaptivecards-designer.ts * Convey Ctrl+M shortcut key to move out of code editor (#7072) * provide hint about ctrl+m toggling tab behavior * rephrase Co-authored-by: Pankaj Bhojwani <[email protected]> Co-authored-by: Canhua Li <[email protected]> * fixed same merge conflict in another commit * color fix (#7081) Co-authored-by: Pankaj Bhojwani <[email protected]> * update headers (#7082) * Add title to iframe component in December 2019 blog post (#7083) * add title * spacing Co-authored-by: Pankaj Bhojwani <[email protected]> * change text color (#7091) Co-authored-by: Pankaj Bhojwani <[email protected]> * add aria label (#7090) Co-authored-by: Pankaj Bhojwani <[email protected]> * update tab indices accordingly (#7119) Co-authored-by: Vsevolod <[email protected]> * Adjust contrast on designer peer buttons (#7110) * Update role of 'Select Host App' combobox (#7118) Co-authored-by: root <[email protected]> Co-authored-by: Vsevolod <[email protected]> * Delete old samples that don't have latest features (causing accessibility issues) (#7106) * updates to samples * phone number regex * delete files instead Co-authored-by: Pankaj Bhojwani <[email protected]> Co-authored-by: Vsevolod <[email protected]> * removed carousel samples * fixed merge conflict line 8 of package.json * fix titles (#7130) Co-authored-by: Pankaj Bhojwani <[email protected]> * fixed merge conflict in ADCIOSVisualizerUITests * add tooltip on initialization (#7126) * more titles to iframes (#7135) Co-authored-by: Pankaj Bhojwani <[email protected]> * add alt text (#7131) Co-authored-by: Pankaj Bhojwani <[email protected]> * use active border instead of white (#7122) * add landmark (#7145) * Use HTML lists for hyperlinks (#7127) * Use HTML lists for hyperlinks * consistent spacing in css file Co-authored-by: root <[email protected]> Co-authored-by: Vsevolod <[email protected]> Co-authored-by: PankajBhojwani <[email protected]> * Add a caption to the table present in the 'help' dialog (#7149) * move keyboard shortcut text to caption * use innerText instead * don't let focus fall on card designer surface (#7153) * add 1 px left margin (#7142) * Add underline styling to hyperlinks (#7157) Co-authored-by: root <[email protected]> Co-authored-by: Vsevolod <[email protected]> * Allow footer hyperlinks to wrap (#7147) Co-authored-by: PankajBhojwani <[email protected]> * Website updates (#7163) * Changed teams target version to 1.4 * Added october and november blog posts * fixed emojis * testing font weights for emoji * removed bold from emojis as not required to render * added blogs posts * fixed video titles * updated roadmap links * removed repeated sentence Co-authored-by: Canhua Li <[email protected]> * [JS] Remove `role='menubar'` and `role='menuitem'` from `ActionCollection` (#6763) Fixes #6425 Fixes #6019 Related #4859 * sync loc file * more package-locks * cherry pick 646503b * return focus after closing (#7155) * final cherry picks for 5e0f39f and 67187d0 * update designer * revert package-lock.json changes Co-authored-by: Rahul Amlekar <[email protected]> Co-authored-by: RahulAmlekar <[email protected]> Co-authored-by: J.P. Roca <[email protected]> Co-authored-by: Siddharth Gulati <[email protected]> Co-authored-by: Paul Campbell <[email protected]> Co-authored-by: almedina-ms <[email protected]> Co-authored-by: Roy Nehoran <[email protected]> Co-authored-by: naramaka <[email protected]> Co-authored-by: PankajBhojwani <[email protected]> Co-authored-by: Pankaj Bhojwani <[email protected]> Co-authored-by: Will Shown <[email protected]> Co-authored-by: Vsevolod <[email protected]> Co-authored-by: Carlos Zamora <[email protected]> Co-authored-by: root <[email protected]>
licanhua
added a commit
that referenced
this pull request
Sep 15, 2022
#7873) * Merge website march22 to main branch (#7857) * Updating Bot Framework links (#6686) Co-authored-by: RahulAmlekar <[email protected]> * Scenario cards + Website indentation fix (#6733) * Changed teams target version to 1.4 * added RTL scenario card * Added scenario card for tooltip and isEnabled * Added order confirmation scenario * Added flight update table scenario * Added restaurant order scenario card * Added app login scenario * Fixed indentation in schema explorer * Testing application login on the samples page * Added extended 1.5 scenario cards * Fixed boolean and duplicate id on cards * added flightupdatetable and restaurant order to ignored cards for .NET tests Co-authored-by: Canhua Li <[email protected]> * Removing "Preview" from Viva Connections (#6791) Co-authored-by: Paul Campbell <[email protected]> * Added blog posts for oct and nov (#6873) * Changed teams target version to 1.4 * Added october and november blog posts * fixed emojis * testing font weights for emoji * removed bold from emojis as not required to render Co-authored-by: Canhua Li <[email protected]> Co-authored-by: almedina-ms <[email protected]> * make columnsets clickable (#7042) Co-authored-by: Canhua Li <[email protected]> * fix merge conflicts with berlin containers in adaptivecards-designer.ts * Convey Ctrl+M shortcut key to move out of code editor (#7072) * provide hint about ctrl+m toggling tab behavior * rephrase Co-authored-by: Pankaj Bhojwani <[email protected]> Co-authored-by: Canhua Li <[email protected]> * fixed same merge conflict in another commit * color fix (#7081) Co-authored-by: Pankaj Bhojwani <[email protected]> * update headers (#7082) * Add title to iframe component in December 2019 blog post (#7083) * add title * spacing Co-authored-by: Pankaj Bhojwani <[email protected]> * change text color (#7091) Co-authored-by: Pankaj Bhojwani <[email protected]> * add aria label (#7090) Co-authored-by: Pankaj Bhojwani <[email protected]> * update tab indices accordingly (#7119) Co-authored-by: Vsevolod <[email protected]> * Adjust contrast on designer peer buttons (#7110) * Update role of 'Select Host App' combobox (#7118) Co-authored-by: root <[email protected]> Co-authored-by: Vsevolod <[email protected]> * Delete old samples that don't have latest features (causing accessibility issues) (#7106) * updates to samples * phone number regex * delete files instead Co-authored-by: Pankaj Bhojwani <[email protected]> Co-authored-by: Vsevolod <[email protected]> * removed carousel samples * fixed merge conflict line 8 of package.json * fix titles (#7130) Co-authored-by: Pankaj Bhojwani <[email protected]> * fixed merge conflict in ADCIOSVisualizerUITests * add tooltip on initialization (#7126) * more titles to iframes (#7135) Co-authored-by: Pankaj Bhojwani <[email protected]> * add alt text (#7131) Co-authored-by: Pankaj Bhojwani <[email protected]> * use active border instead of white (#7122) * add landmark (#7145) * Use HTML lists for hyperlinks (#7127) * Use HTML lists for hyperlinks * consistent spacing in css file Co-authored-by: root <[email protected]> Co-authored-by: Vsevolod <[email protected]> Co-authored-by: PankajBhojwani <[email protected]> * Add a caption to the table present in the 'help' dialog (#7149) * move keyboard shortcut text to caption * use innerText instead * don't let focus fall on card designer surface (#7153) * add 1 px left margin (#7142) * Add underline styling to hyperlinks (#7157) Co-authored-by: root <[email protected]> Co-authored-by: Vsevolod <[email protected]> * Allow footer hyperlinks to wrap (#7147) Co-authored-by: PankajBhojwani <[email protected]> * Website updates (#7163) * Changed teams target version to 1.4 * Added october and november blog posts * fixed emojis * testing font weights for emoji * removed bold from emojis as not required to render * added blogs posts * fixed video titles * updated roadmap links * removed repeated sentence Co-authored-by: Canhua Li <[email protected]> * [JS] Remove `role='menubar'` and `role='menuitem'` from `ActionCollection` (#6763) Fixes #6425 Fixes #6019 Related #4859 * sync loc file * more package-locks * cherry pick 646503b * return focus after closing (#7155) * final cherry picks for 5e0f39f and 67187d0 * update designer * revert package-lock.json changes Co-authored-by: Rahul Amlekar <[email protected]> Co-authored-by: RahulAmlekar <[email protected]> Co-authored-by: J.P. Roca <[email protected]> Co-authored-by: Siddharth Gulati <[email protected]> Co-authored-by: Paul Campbell <[email protected]> Co-authored-by: almedina-ms <[email protected]> Co-authored-by: Roy Nehoran <[email protected]> Co-authored-by: naramaka <[email protected]> Co-authored-by: PankajBhojwani <[email protected]> Co-authored-by: Pankaj Bhojwani <[email protected]> Co-authored-by: Will Shown <[email protected]> Co-authored-by: Vsevolod <[email protected]> Co-authored-by: Carlos Zamora <[email protected]> Co-authored-by: root <[email protected]> * prefix adaptivecards# for ttpolicy (#7860) * remove dead code searchAndRemoveForbiddenElements * Revert "use active border instead of white (#7122)" (#7874) This reverts commit 68d88fe. Co-authored-by: Rahul Amlekar <[email protected]> Co-authored-by: RahulAmlekar <[email protected]> Co-authored-by: J.P. Roca <[email protected]> Co-authored-by: Siddharth Gulati <[email protected]> Co-authored-by: Paul Campbell <[email protected]> Co-authored-by: almedina-ms <[email protected]> Co-authored-by: Roy Nehoran <[email protected]> Co-authored-by: naramaka <[email protected]> Co-authored-by: PankajBhojwani <[email protected]> Co-authored-by: Pankaj Bhojwani <[email protected]> Co-authored-by: Will Shown <[email protected]> Co-authored-by: Vsevolod <[email protected]> Co-authored-by: Carlos Zamora <[email protected]> Co-authored-by: root <[email protected]>
michaelfarnsworth
pushed a commit
to michaelfarnsworth/AdaptiveCards
that referenced
this pull request
Nov 10, 2022
…lity issues) (microsoft#7106) * updates to samples * phone number regex * delete files instead Co-authored-by: Pankaj Bhojwani <[email protected]> Co-authored-by: Vsevolod <[email protected]>
michaelfarnsworth
pushed a commit
to michaelfarnsworth/AdaptiveCards
that referenced
this pull request
Nov 10, 2022
* Updating Bot Framework links (microsoft#6686) Co-authored-by: RahulAmlekar <[email protected]> * Scenario cards + Website indentation fix (microsoft#6733) * Changed teams target version to 1.4 * added RTL scenario card * Added scenario card for tooltip and isEnabled * Added order confirmation scenario * Added flight update table scenario * Added restaurant order scenario card * Added app login scenario * Fixed indentation in schema explorer * Testing application login on the samples page * Added extended 1.5 scenario cards * Fixed boolean and duplicate id on cards * added flightupdatetable and restaurant order to ignored cards for .NET tests Co-authored-by: Canhua Li <[email protected]> * Removing "Preview" from Viva Connections (microsoft#6791) Co-authored-by: Paul Campbell <[email protected]> * Added blog posts for oct and nov (microsoft#6873) * Changed teams target version to 1.4 * Added october and november blog posts * fixed emojis * testing font weights for emoji * removed bold from emojis as not required to render Co-authored-by: Canhua Li <[email protected]> Co-authored-by: almedina-ms <[email protected]> * make columnsets clickable (microsoft#7042) Co-authored-by: Canhua Li <[email protected]> * fix merge conflicts with berlin containers in adaptivecards-designer.ts * Convey Ctrl+M shortcut key to move out of code editor (microsoft#7072) * provide hint about ctrl+m toggling tab behavior * rephrase Co-authored-by: Pankaj Bhojwani <[email protected]> Co-authored-by: Canhua Li <[email protected]> * fixed same merge conflict in another commit * color fix (microsoft#7081) Co-authored-by: Pankaj Bhojwani <[email protected]> * update headers (microsoft#7082) * Add title to iframe component in December 2019 blog post (microsoft#7083) * add title * spacing Co-authored-by: Pankaj Bhojwani <[email protected]> * change text color (microsoft#7091) Co-authored-by: Pankaj Bhojwani <[email protected]> * add aria label (microsoft#7090) Co-authored-by: Pankaj Bhojwani <[email protected]> * update tab indices accordingly (microsoft#7119) Co-authored-by: Vsevolod <[email protected]> * Adjust contrast on designer peer buttons (microsoft#7110) * Update role of 'Select Host App' combobox (microsoft#7118) Co-authored-by: root <[email protected]> Co-authored-by: Vsevolod <[email protected]> * Delete old samples that don't have latest features (causing accessibility issues) (microsoft#7106) * updates to samples * phone number regex * delete files instead Co-authored-by: Pankaj Bhojwani <[email protected]> Co-authored-by: Vsevolod <[email protected]> * removed carousel samples * fixed merge conflict line 8 of package.json * fix titles (microsoft#7130) Co-authored-by: Pankaj Bhojwani <[email protected]> * fixed merge conflict in ADCIOSVisualizerUITests * add tooltip on initialization (microsoft#7126) * more titles to iframes (microsoft#7135) Co-authored-by: Pankaj Bhojwani <[email protected]> * add alt text (microsoft#7131) Co-authored-by: Pankaj Bhojwani <[email protected]> * use active border instead of white (microsoft#7122) * add landmark (microsoft#7145) * Use HTML lists for hyperlinks (microsoft#7127) * Use HTML lists for hyperlinks * consistent spacing in css file Co-authored-by: root <[email protected]> Co-authored-by: Vsevolod <[email protected]> Co-authored-by: PankajBhojwani <[email protected]> * Add a caption to the table present in the 'help' dialog (microsoft#7149) * move keyboard shortcut text to caption * use innerText instead * don't let focus fall on card designer surface (microsoft#7153) * add 1 px left margin (microsoft#7142) * Add underline styling to hyperlinks (microsoft#7157) Co-authored-by: root <[email protected]> Co-authored-by: Vsevolod <[email protected]> * Allow footer hyperlinks to wrap (microsoft#7147) Co-authored-by: PankajBhojwani <[email protected]> * Website updates (microsoft#7163) * Changed teams target version to 1.4 * Added october and november blog posts * fixed emojis * testing font weights for emoji * removed bold from emojis as not required to render * added blogs posts * fixed video titles * updated roadmap links * removed repeated sentence Co-authored-by: Canhua Li <[email protected]> * [JS] Remove `role='menubar'` and `role='menuitem'` from `ActionCollection` (microsoft#6763) Fixes microsoft#6425 Fixes microsoft#6019 Related microsoft#4859 * sync loc file * more package-locks * cherry pick 646503b * return focus after closing (microsoft#7155) * final cherry picks for 5e0f39f and 67187d0 * update designer * revert package-lock.json changes Co-authored-by: Rahul Amlekar <[email protected]> Co-authored-by: RahulAmlekar <[email protected]> Co-authored-by: J.P. Roca <[email protected]> Co-authored-by: Siddharth Gulati <[email protected]> Co-authored-by: Paul Campbell <[email protected]> Co-authored-by: almedina-ms <[email protected]> Co-authored-by: Roy Nehoran <[email protected]> Co-authored-by: naramaka <[email protected]> Co-authored-by: PankajBhojwani <[email protected]> Co-authored-by: Pankaj Bhojwani <[email protected]> Co-authored-by: Will Shown <[email protected]> Co-authored-by: Vsevolod <[email protected]> Co-authored-by: Carlos Zamora <[email protected]> Co-authored-by: root <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Related Issue
Resolves 35357399 and 35368881, tracked in #5655
Description
Update
Inputs.jsonandInputForm.jsonto use new features likelabelanderrorMessage, to close the issues listed in #5655Delete the samples
Inputs.jsonandInputForm.json, because we already have the samplesInputFormWithLabels.jsonandInputsWithValidation.json, which solve the issues mentionedHow Verified
Files no longer exist so neither do the issues with them