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

PR for #41 & #43 - Implement Color and Colors #44

Merged
merged 3 commits into from
Nov 5, 2018
Merged

Conversation

jamesmcroft
Copy link
Member

This PR includes a new project, XPlat.UI, which contains a wrapper for the Windows Color and Colors classes.

This resolves #41 and #43.

This PR also includes a change to remove StyleCop in preference of post-build step with SonarQube. Also included is a change to take advantage of SourceLink.

@tom-made tom-made self-requested a review November 5, 2018 07:53

#if __ANDROID__

public static implicit operator Color(Android.Graphics.Color color)
Copy link
Collaborator

Choose a reason for hiding this comment

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

This clears up my comments around platform specifics.

Copy link
Collaborator

@tom-made tom-made left a comment

Choose a reason for hiding this comment

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

I'll approve these changes and merge the PR. Will send an e-mail later on, need to ask a question around the docs and samples.

@tom-made tom-made merged commit cc4d7d4 into develop Nov 5, 2018
@tom-made tom-made deleted the color-api branch November 5, 2018 07:55
This was referenced Nov 5, 2018
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.

Implement Windows.UI.Color
2 participants