-
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
[UWP] Add rendering support for tooltip property #5763
Conversation
rendering test since the feature isn't developed for iOS
…iveCards into rebecch/uwptooltiprender
Thanks @jwoo-msft! |
@@ -0,0 +1,225 @@ | |||
{ |
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.
nit: it would be great to have a screenshot of this for UWP.
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.
Screenshot's not going to help too much here as the interesting behavior is in the tooltips and the accessibility text, neither of which will be captured by that. Maybe I can update the card to better describe the expected behavior for each case.
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.
Just one question but LGTM otherwise
* [UWP] Add rendering support for tooltip property * [iOS] added TooltipTestCard.json to a exclusion list for parsing & rendering test since the feature isn't developed for iOS * Add SAL * Add better descriptions to test card Co-authored-by: nesalang <Joseph.Woo@microsoft.com>
* [UWP] Add rendering support for tooltip property * [iOS] added TooltipTestCard.json to a exclusion list for parsing & rendering test since the feature isn't developed for iOS * Add SAL * Add better descriptions to test card Co-authored-by: nesalang <Joseph.Woo@microsoft.com>
Related Issue
Fixes #5470
Description
Implemented UWP support for tooltips. Added logic to handle correctly setting the title and tooltip properties to the UI tooltip and automation name and description. Refactored automation name/description setting and tooltip creation for easier code sharing.
This PR does not fix related issue #5579. This means that container contents of a container are still not read in the case where the title is not present.
Sample Card
Added new card at samples\v1.5\Test\TooltipTestCard.json
How Verified
Using the above test card:
Microsoft Reviewers: Open in CodeFlow