Skip to content

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