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

ImageEx CornerRadius RS3 Support and RoundImageEx Obsolete Notice #1586

Merged
merged 4 commits into from
Nov 2, 2017

Conversation

michael-hawker
Copy link
Member

Issue: #1391

PR Type

What kind of change does this PR introduce?

[ x ] Bugfix
[ x ] Feature
[ ] Code style update (formatting)
[ x ] Refactoring (no functional changes, no api changes)
[ ] Build or CI related changes
[ x ] Documentation content changes
[ ] Sample app changes
[ x ] Other... Please describe:  Deprecation

What is the current behavior?

Round Image uses its own custom brush and behaves differently than Image/ImageEx in layout (Issue #1391)

PR Checklist

Please check if your PR fulfills the following requirements:

  • [ x ] Tested code with current supported SDKs
  • [ x ] Docs have been added / updated (for bug fixes / features)
  • [ x ] Sample in sample app has been added / updated (for bug fixes / features)
  • [ x ] Tests for the changes have been added (for bug fixes / features) (if applicable)

What is the new behavior?

This adds a CornerRadius property to ImageEx to replace the need for RoundImageEx entirely, now that RS3 properly supports the CornerRadius property.

Does this PR introduce a breaking change?

[ ] Yes
[ x ] No*

Does mark RoundImageEx as Obsolete.

Other information

If trying to use new CornerRadius on ImageEx on down-level platforms, rendering is not as expected, as if had tried to do the same on a Grid surrounding an Image.

…ues per corner) and fixes for parallel compatibility with RoundImageEx for down-level.
Copy link
Contributor

@nmetulev nmetulev left a comment

Choose a reason for hiding this comment

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

Looks great, only have a small request to change the wording in the sample from RS3 to Fall Creators Update

CornerRadius="30,60,30,60"
Style="{StaticResource RectangleStyle}"/>

<TextBlock Text="Above ImageEx will have rounded corners on RS3." Margin="4" HorizontalAlignment="Center"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you change this from RS3 to Fall Creators Update?

@nmetulev nmetulev added this to the v2.1 milestone Nov 1, 2017
@michael-hawker
Copy link
Member Author

@nmetulev updated wording, so we should be all set now.

@nmetulev nmetulev merged commit 7c9f445 into CommunityToolkit:master Nov 2, 2017
@WilliamABradley
Copy link
Contributor

@nmetulev So you won't actually remove the Code for RoundImageEx, until Fall Creators Update becomes Minimum Supported Version of the UWP Community Toolkit? It just has the Obsolete flag?

@nmetulev
Copy link
Contributor

nmetulev commented Nov 2, 2017

Yeah, we won't remove it until a future major update

@WilliamABradley
Copy link
Contributor

Future Major update of the Toolkit, or Windows? Having RoundImageEx is helpful for Backwards Compat.

@nmetulev
Copy link
Contributor

nmetulev commented Nov 2, 2017

Agree with you. Need to make sure we don't create any breaking changes and continue support our min version. Breaking changes only happen on major releases of the toolkit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants