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

Image Control #517

Merged
merged 6 commits into from
Feb 6, 2023
Merged

Image Control #517

merged 6 commits into from
Feb 6, 2023

Conversation

Aybex
Copy link
Contributor

@Aybex Aybex commented Jan 30, 2023

Implements a ui:Image Control with CornerRadius, BorderThickness, BorderBrush and Background

Pull request type

Please check the type of change your PR introduces:

  • Update
  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes

What is the current behavior?

No easy way for an image with rounded corners.

What is the new behavior?

  • ui:Image gives access all Image properties with border and corner properties.
    For Transparent Images, you can set Background to fill the transparency

Other information

image

@github-actions github-actions bot added controls Changes to the appearance or logic of custom controls. styles Topic is related to styles labels Jan 30, 2023
@@ -41,10 +41,13 @@
<controls:GalleryControlPresenter
Grid.Row="0"
Margin="0"
CodeText="&lt;Image Height=&quot;100&quot; Source=&quot;Assets\MyImage.jpg&quot; /&gt;"
HeaderText="A basic Image from a local file">
Copy link
Member

Choose a reason for hiding this comment

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

I'd recommend leaving the default Image and adding a second one below, ui:Image, to show that there is a choice.

Copy link
Member

@pomianowski pomianowski left a comment

Choose a reason for hiding this comment

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

It would be better to expand the Gallery, not replace the currently shown example

@pomianowski pomianowski self-assigned this Feb 3, 2023
@Aybex
Copy link
Contributor Author

Aybex commented Feb 5, 2023

Added both Image and ui:Image to The Gallery

@pomianowski pomianowski merged commit b5c2601 into lepoco:development Feb 6, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
controls Changes to the appearance or logic of custom controls. styles Topic is related to styles
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants