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

[WPF][InputEvolution] Fix fallback and error messages in non interactive scenarios #4502

Merged
merged 2 commits into from
Jul 31, 2020

Conversation

almedina-ms
Copy link
Contributor

@almedina-ms almedina-ms commented Jul 30, 2020

Related Issue

Fixes #4474

Description

This PR includes two fixes:

  1. Add validation when there is no rendereable element (this means that the fallback results in dropping the element)
  2. Add validation to avoid setting error messages to inputs when interactivity is disabled. Due to the nature of this property, it doesn't matter if the inputs are not linked to error messages

How Verified

  1. The fix was verified by executing the ./RunAllTests.ps1 script
  2. The fix was also verified manually as can be seen below

SimpleFallback.json
image

Inputs (v1.3) with WindowsTimeline host config
image

Microsoft Reviewers: Open in CodeFlow

@shalinijoshi19
Copy link
Member

@almedina-ms tx! To confirm, you were able to test this with the 1.2 version of the sample(which is what had been failing? )

Copy link
Member

@jwoo-msft jwoo-msft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@shalinijoshi19
Copy link
Member

shalinijoshi19 commented Jul 31, 2020

@almedina-ms there are more than test changes here. We should update the description/title to be reflective of that(its a bit confusing right now with "Fux WPF test errors")

@almedina-ms almedina-ms changed the title [WPF][InputEvolution] Fix WPF tests errors [WPF][InputEvolution] Fix fallback and error messages in non interactive scenarios Jul 31, 2020
@almedina-ms
Copy link
Contributor Author

@almedina-ms tx! To confirm, you were able to test this with the 1.2 version of the sample(which is what had been failing? )

The bug was found with the SimpleFallback card. As stated in the description that card works now

@almedina-ms almedina-ms merged commit bee56e9 into main Jul 31, 2020
@almedina-ms almedina-ms deleted the user/almedina-ms/IE_WPF_FixTestError branch July 31, 2020 18:12
almedina-ms added a commit that referenced this pull request Jul 31, 2020
* Fix Fallback and error message with no interactivity (#4502)

* Remove function that should not be used by customers

Co-authored-by: Paul Campbell <paulcam@microsoft.com>
almedina-ms added a commit that referenced this pull request Jul 31, 2020
* Fix Fallback and error message with no interactivity (#4502)

* Remove function that should not be used by customers

Co-authored-by: Paul Campbell <paulcam@microsoft.com>
almedina-ms added a commit that referenced this pull request Jul 31, 2020
* Fix Fallback and error message with no interactivity (#4502)

* Remove unused, untested and consumable function (#4507)

* Fix Fallback and error message with no interactivity (#4502)

* Remove function that should not be used by customers

Co-authored-by: Paul Campbell <paulcam@microsoft.com>

Co-authored-by: Paul Campbell <paulcam@microsoft.com>
shalinijoshi19 added a commit that referenced this pull request Aug 3, 2020
* Remove unused, untested and consumable function (#4507)

* Fix Fallback and error message with no interactivity (#4502)

* Remove function that should not be used by customers

Co-authored-by: Paul Campbell <paulcam@microsoft.com>

* Updating version to 2.0.0 for nuget release

Co-authored-by: almedina-ms <35784165+almedina-ms@users.noreply.github.com>
Co-authored-by: Paul Campbell <paulcam@microsoft.com>
shalinijoshi19 added a commit that referenced this pull request Aug 3, 2020
* Remove unused, untested and consumable function (#4507)

* Fix Fallback and error message with no interactivity (#4502)

* Remove function that should not be used by customers

Co-authored-by: Paul Campbell <paulcam@microsoft.com>

* Updating version to 2.0.0 for nuget release

Co-authored-by: almedina-ms <35784165+almedina-ms@users.noreply.github.com>
Co-authored-by: Paul Campbell <paulcam@microsoft.com>
rankush pushed a commit to rankush/AdaptiveCards that referenced this pull request May 8, 2024
* Fix Fallback and error message with no interactivity (microsoft#4502)

* Remove function that should not be used by customers

Co-authored-by: Paul Campbell <paulcam@microsoft.com>
rankush pushed a commit to rankush/AdaptiveCards that referenced this pull request May 8, 2024
* Remove unused, untested and consumable function (microsoft#4507)

* Fix Fallback and error message with no interactivity (microsoft#4502)

* Remove function that should not be used by customers

Co-authored-by: Paul Campbell <paulcam@microsoft.com>

* Updating version to 2.0.0 for nuget release

Co-authored-by: almedina-ms <35784165+almedina-ms@users.noreply.github.com>
Co-authored-by: Paul Campbell <paulcam@microsoft.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.

[WPF][Input evolution]SimpleFallback.json scenario card fails to render with the new WPF renderer
3 participants