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] Update high contrast colors | MicaAlt colors - update high contrast colors #6787

Merged
merged 11 commits into from
Mar 4, 2022

Conversation

tashatitova
Copy link
Contributor

Did a high contrast pass and fixed some bugs on tabs, selected tab, outlines, tab close button.
Added a border for the close button in high contrast as usually required by accessibility.

Updated MicaAlt colors to proper background in high contrast as these are usually backgrounds and we want to avoid flooding large areas with highlight color.

Before:
before

Updated, multiple hc themes:

TabView-after.mp4

@ghost ghost added the needs-triage Issue needs to be triaged by the area owners label Mar 4, 2022
@tashatitova
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@beervoley
Copy link
Contributor

:shipit:

…g on press on tab vs press on close button - fixed now
@tashatitova
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@karkarl
Copy link
Contributor

karkarl commented Mar 4, 2022

        <StaticResource x:Key="TabViewItemHeaderCloseButtonBorderBrush" ResourceKey="SystemControlTransparentBrush" />

This should be SubtleFillColorTransparentBrush as part of the standard to avoid using System* brushes for light and dark themes.


Refers to: dev/TabView/TabView_themeresources.xaml:128 in fbfd705. [](commit_id = fbfd705, deletion_comment = False)

Copy link
Contributor

@karkarl karkarl left a comment

Choose a reason for hiding this comment

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

Move Thickness Resources + update transparent brush in light/dark theme.

@tashatitova tashatitova requested a review from karkarl March 4, 2022 22:39
@tashatitova tashatitova merged commit 6fb99c9 into main Mar 4, 2022
@tashatitova tashatitova deleted the user/tatito/tab branch March 4, 2022 22:41
@StephenLPeters StephenLPeters added accessibility Narrator, keyboarding, UIA, etc area-TabView team-Controls Issue for the Controls team and removed needs-triage Issue needs to be triaged by the area owners labels Mar 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility Narrator, keyboarding, UIA, etc area-TabView team-Controls Issue for the Controls team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants