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

[iOS] Add support for colors with alpha #561

Conversation

felix-ht
Copy link
Collaborator

@felix-ht felix-ht commented Mar 3, 2021

Currently the iOS version only supports RGB colors such as #ffee00. Android also supports colors with alphas such as #99ffee00 (where 99 would be the alpha value)
The Purpose of this pull request is to add the support to iOS as well.

The Syntax of the color string for iOS can either follow the old format of #rrggbb or the new format #aarrggbb. Thus this change is downwards compatible and should not break any existing code.
#aarrggbb is consistent with the android implementation.

@felix-ht felix-ht changed the title Add ios support for colors with alpha [iOS] Add support for colors with alpha Mar 3, 2021
@felix-ht
Copy link
Collaborator Author

felix-ht commented Mar 3, 2021

note that colors with alpha in android is currently trick to use as well see #562.

The issues with the pipline seem to be unrealted to my changes. Maybe caused by the flutter 2.0 release?

Copy link
Collaborator

@shroff shroff left a comment

Choose a reason for hiding this comment

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

LGTM!

@tobrun
Copy link
Collaborator

tobrun commented Jun 5, 2021

Thank you for the contribution @felix-ht!
Reran the bots as part of another branch in #642 and everything worked.
Closing as it has landed on master branch.

@tobrun tobrun closed this Jun 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants