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

TabView: Add tooltips to its scrolling buttons. #2369

Conversation

Felix-Dev
Copy link
Contributor

Description

This PR adds tooltips to the TabView's tab list scrolling buttons. Note: Not yet localized.

Motivation and Context

Closes #2345.

How Has This Been Tested?

Tested visually.

Screenshots

image

image

@msft-github-bot msft-github-bot added the needs-triage Issue needs to be triaged by the area owners label May 1, 2020
@ranjeshj ranjeshj requested a review from stmoy May 1, 2020 21:06
@ranjeshj ranjeshj added team-Controls Issue for the Controls team and removed needs-triage Issue needs to be triaged by the area owners labels May 1, 2020
Copy link
Contributor

@stmoy stmoy left a comment

Choose a reason for hiding this comment

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

Look ok to me. Should any tests be added?

@ranjeshj
Copy link
Contributor

ranjeshj commented May 4, 2020

@Felix-Dev Can you please add a test ? Thanks.

@Felix-Dev
Copy link
Contributor Author

Added an interaction test.

@ranjeshj
Copy link
Contributor

ranjeshj commented May 4, 2020

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@StephenLPeters StephenLPeters left a comment

Choose a reason for hiding this comment

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

:shipit:

@StephenLPeters StephenLPeters merged commit d14b639 into microsoft:master May 7, 2020
@Felix-Dev Felix-Dev deleted the user/Felix-Dev/tabview-scrollbutton-tooltip branch May 7, 2020 09:41
ghost pushed a commit to microsoft/terminal that referenced this pull request Jul 7, 2020
See: https://github.com/microsoft/microsoft-ui-xaml/releases/tag/v2.5.0-prerelease.200609001

> ### Notable Changes:
> 
>     Resize tab view items only once the pointer has left the TabViewItem strip (microsoft/microsoft-ui-xaml#2569)
>     Align TabView visuals with Edge (microsoft/microsoft-ui-xaml#2201)
>     Fix background of MenuFlyout in white high contrast (microsoft/microsoft-ui-xaml#2446)
>     TabView: Make TabViewItem consume the TabViewItemHeaderForeground theme resource (microsoft/microsoft-ui-xaml#2348)
>     TabView: Add tooltips to its scrolling buttons. (microsoft/microsoft-ui-xaml#2369)


* [x] Related to #5360 (@jtippet confirms that this alone does not close it.)
* [x] I work here
donno2048 added a commit to donno2048/terminal that referenced this pull request Sep 28, 2020
See: https://github.com/microsoft/microsoft-ui-xaml/releases/tag/v2.5.0-prerelease.200609001

> ### Notable Changes:
> 
>     Resize tab view items only once the pointer has left the TabViewItem strip (microsoft/microsoft-ui-xaml#2569)
>     Align TabView visuals with Edge (microsoft/microsoft-ui-xaml#2201)
>     Fix background of MenuFlyout in white high contrast (microsoft/microsoft-ui-xaml#2446)
>     TabView: Make TabViewItem consume the TabViewItemHeaderForeground theme resource (microsoft/microsoft-ui-xaml#2348)
>     TabView: Add tooltips to its scrolling buttons. (microsoft/microsoft-ui-xaml#2369)


* [x] Related to #5360 (@jtippet confirms that this alone does not close it.)
* [x] I work here
@ghost
Copy link

ghost commented Dec 4, 2020

🎉Microsoft.UI.Xaml v2.5.0 has been released which incorporates this pull request.:tada:

Handy links:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-TabView team-Controls Issue for the Controls team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Proposal/Question: Add Tooltips to TabView's scroll buttons
5 participants