Skip to content
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

[Android] Fix rendering of inputs with no labels #4995

Merged
merged 4 commits into from
Oct 29, 2020

Conversation

almedina-ms
Copy link
Contributor

@almedina-ms almedina-ms commented Oct 29, 2020

Related Issue

Fixes #4994

Description

Adds a null check validation to stop assuming all inputs have a label for setting accessibility properties, during creation and validation

How Verified

The fix was verified using the linked card giving this result, also, other input types were tested

image

Microsoft Reviewers: Open in CodeFlow

@almedina-ms almedina-ms changed the title Fix rendering of inputs with no labels [Android] Fix rendering of inputs with no labels Oct 29, 2020
@almedina-ms almedina-ms merged commit 6ef92a7 into main Oct 29, 2020
@almedina-ms almedina-ms deleted the user/almedina-ms/AndroidMissingLabelFix branch October 29, 2020 21:13
almedina-ms added a commit that referenced this pull request Oct 29, 2020
* Fix rendering of inputs with no labels

* Fix error messages for missing labels too

Co-authored-by: Risheek Rajolu <risheekrr@gmail.com>
almedina-ms added a commit that referenced this pull request Oct 29, 2020
* [Android] Switch pipelines to use Windows hosted pools (#4987)

* Use Microsoft's Windows hosted agent pools

* .bat extension for Windows

* Explicit pool name

* Remove sudo, because Windows

* Windows-style path, because Windows

* Trying with cmd script

* Fix path

* syntax

* Disable get ndk

* daily too

* [Samples][Accessibility] TextBlock wrap in Scenario cards (#4991)

* Text: wrap for scenario samples

* Copyright, license

* More wrap

* Missed some

* Spaces for indentation

Co-authored-by: Shalini Joshi-MSFT <shalinij@microsoft.com>

* [Android] Fix rendering of inputs with no labels (#4995)

* Fix rendering of inputs with no labels

* Fix error messages for missing labels too

Co-authored-by: Risheek Rajolu <risheekrr@gmail.com>

Co-authored-by: Risheek Rajolu <risheekrr@gmail.com>
Co-authored-by: Shalini Joshi-MSFT <shalinij@microsoft.com>
rankush pushed a commit to rankush/AdaptiveCards that referenced this pull request May 8, 2024
* Fix rendering of inputs with no labels

* Fix error messages for missing labels too

Co-authored-by: Risheek Rajolu <risheekrr@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Android][ChoiceSet Input] [Choiceset input (required) rendering fails inside a container]
2 participants