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

Remove Maui Compatibility #2

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Remove Maui Compatibility #2

wants to merge 3 commits into from

Conversation

vryan95
Copy link

@vryan95 vryan95 commented Nov 2, 2024

Removed the MAUI Compatibility stuff. Also did refactoring for readability

  • Changed RelativeLayout to use Grid
  • Changed LowerChild to use ZIndex instead
  • Added regions in SwipeCardView.cs
  • Moved SwipeCardDirectionExtensions to its own file

Updated `Microsoft.Maui.Controls` to version `8.0.92` and
`Microsoft.Extensions.Logging.Debug` to version `8.0.1` in
project files. Added `Microsoft.Maui.Controls.Compatibility`
package reference. Reorganized `SwipeCardView.cs` with regions
for better code structure. Moved `OnPanUpdated` to `Event
Handlers` region and added a new method for pan gesture updates.
Removed `DraggingCardPosition` property, initializing it in the
constructor. Moved `SwipeCardDirectionExtensions` to a separate
file and cleaned up redundant code and `using` directives.
Removed UseSwipeCardView extension method from MauiProgram.cs.
Deleted AppBuilderExtensions.cs, which contained the extension method.
Updated SwipeCardView to use Grid instead of RelativeLayout.
Removed Microsoft.Maui.Controls.Compatibility namespace import.
Revised card stacking logic to iterate in reverse order.
Changed card addition to use Grid.Children.Add.
Implemented z-index management using Grid's ZIndex property.
Adjusted swipe handling to modify ZIndex of the top card.
@vryan95
Copy link
Author

vryan95 commented Nov 2, 2024

#1

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.

1 participant