-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Port Image.Aspect #808
Closed
Closed
Port Image.Aspect #808
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Already exists an Aspect enum in Microsoft.Maui.Controls, but I believe that is a legacy reference.
This is necessary to implement the handlers.
This reverts commit 1e44c151a4c621566ea055a77b5ad89c1af5fd32.
I move the Aspect enum to core project because it is not reference the core.controls project
I believe that pull request #759 implements the aspect functionality and my pull request can be closed. Someone can confirm this? |
Thanks for this PR @ionixjunior. Seems there might have been some mis-communication on this issue. Sorry about that. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of Change
Implements #400
Additions made
PR Checklist
Doubts
Original reference of Aspect enum
The Aspect enum already existed inside the
Controls.Core
project, so I needed to move this implementation to theCore
project. It is right?Custom ImageView implementations
There are custom implementations of ImageView inside Android and iOS renderers. These classes are
FormsImageView
andFormsUIImageView
. I didn't use them to specify in theImageHandler
, but I believe that should be important to do this. Should I use these custom ImageView implementations? Should I move these implementations toCore
project?ImageView Android implementation
The original implementation of Android custom renderer uses the
ImageView
class ofAndroid.Widget
. However, in the handler implementation, I usedAppCompatImageView
class ofAndroidX.AppCompat.Widget
. I believe that is the best way, but I was in doubt if I need to maintain the original implementation. Should I use theAppCompatImageView
?