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

Fix Provided consistent behavior of text color in tabbed page in all platforms. #24782

Merged
merged 3 commits into from
Sep 26, 2024

Conversation

devanathan-vaithiyanathan
Copy link
Contributor

@devanathan-vaithiyanathan devanathan-vaithiyanathan commented Sep 16, 2024

Root cause

The unSelectedBarTextColor is passed as null by default. This causes the tint color of the UITabbar to be applied. As a result, unSelectedBarTextColor on the iOS and Mac platforms is a different color.

Description of Change

We can pass the BarTextColor property to the unSelectedBarTextColor instead of null.

Validated the behaviour in the following platforms

  • Android
  • Windows
  • iOS
  • Mac

Output Screenshot

Before After
MAC
MAC
iOS
iOS

@jfversluis
Copy link
Member

jfversluis commented Sep 24, 2024

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

1 similar comment
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@PureWeen
Copy link
Member

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@PureWeen
Copy link
Member

/rebase

@PureWeen
Copy link
Member

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@PureWeen PureWeen enabled auto-merge (squash) September 25, 2024 23:09
@PureWeen
Copy link
Member

I checked the device logs and it looks like iOS controls is locking up from HybridWebView tests unrelated to this PR

image

@PureWeen PureWeen merged commit f23bacf into dotnet:net9.0 Sep 26, 2024
119 of 121 checks passed
@samhouts samhouts added the fixed-in-net9.0-nightly This may be available in a nightly release! label Oct 1, 2024
@AlleSchonWeg
Copy link

@PureWeen
Is it possible to bring this fix to .Net 8 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-controls-tabbedpage TabbedPage community ✨ Community Contribution fixed-in-net9.0-nightly This may be available in a nightly release!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants