Skip to content

Migrate EmployeeDirectory and WorkingWithGestures samples#641

Merged
jsuarezruiz merged 11 commits intodotnet:mainfrom
morning4coffe-dev:dev/doti/employee-directory-mgt
Sep 17, 2025
Merged

Migrate EmployeeDirectory and WorkingWithGestures samples#641
jsuarezruiz merged 11 commits intodotnet:mainfrom
morning4coffe-dev:dev/doti/employee-directory-mgt

Conversation

@morning4coffe-dev
Copy link
Contributor

@morning4coffe-dev morning4coffe-dev commented Jul 16, 2025

closes #614 and closes #635

Migrates and improves the original EmployeeDirectory and WorkingWithGestures from Xamarin to .NET MAUI.

@morning4coffe-dev
Copy link
Contributor Author

This is an AI migration, manual TODO: Adjust design, refactor, change screenshots

@morning4coffe-dev morning4coffe-dev changed the title Migrate EmployeeDirectory sample Migrate EmployeeDirectory and WorkingWithGestures samples Jul 16, 2025
@jsuarezruiz jsuarezruiz self-requested a review July 17, 2025 14:01
@jsuarezruiz jsuarezruiz self-assigned this Jul 17, 2025
@jsuarezruiz jsuarezruiz marked this pull request as ready for review July 22, 2025 14:41
Copy link
Contributor

@jsuarezruiz jsuarezruiz left a comment

Choose a reason for hiding this comment

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

  Warning: /Users/runner/work/maui-samples/maui-samples/9.0/UserInterface/WorkingWithGestures/TapGesture/TapGesture/TapInsideImage.cs(42,35): warning CS8622: Nullability of reference types in type of parameter 'sender' of 'void TapInsideImage.OnTapGestureRecognizerTapped(object sender, EventArgs args)' doesn't match the target delegate 'EventHandler<TappedEventArgs>' (possibly because of nullability attributes). [/Users/runner/work/maui-samples/maui-samples/9.0/UserInterface/WorkingWithGestures/TapGesture/TapGesture/TapGesture.csproj::TargetFramework=net9.0-ios]
  Warning: /Users/runner/work/maui-samples/maui-samples/9.0/UserInterface/WorkingWithGestures/TapGesture/TapGesture/TapInsideFrame.cs(45,35): warning CS8622: Nullability of reference types in type of parameter 'sender' of 'void TapInsideFrame.OnTapGestureRecognizerTapped(object sender, EventArgs args)' doesn't match the target delegate 'EventHandler<TappedEventArgs>' (possibly because of nullability attributes). [/Users/runner/work/maui-samples/maui-samples/9.0/UserInterface/WorkingWithGestures/TapGesture/TapGesture/TapGesture.csproj::TargetFramework=net9.0-ios]
  Warning: /Users/runner/work/maui-samples/maui-samples/9.0/UserInterface/WorkingWithGestures/TapGesture/TapGesture/TapInsideImage.cs(42,35): warning CS8622: Nullability of reference types in type of parameter 'sender' of 'void TapInsideImage.OnTapGestureRecognizerTapped(object sender, EventArgs args)' doesn't match the target delegate 'EventHandler<TappedEventArgs>' (possibly because of nullability attributes). [/Users/runner/work/maui-samples/maui-samples/9.0/UserInterface/WorkingWithGestures/TapGesture/TapGesture/TapGesture.csproj::TargetFramework=net9.0-android]
  Warning: /Users/runner/work/maui-samples/maui-samples/9.0/UserInterface/WorkingWithGestures/TapGesture/TapGesture/TapInsideFrame.cs(45,35): warning CS8622: Nullability of reference types in type of parameter 'sender' of 'void TapInsideFrame.OnTapGestureRecognizerTapped(object sender, EventArgs args)' doesn't match the target delegate 'EventHandler<TappedEventArgs>' (possibly because of nullability attributes). [/Users/runner/work/maui-samples/maui-samples/9.0/UserInterface/WorkingWithGestures/TapGesture/TapGesture/TapGesture.csproj::TargetFramework=net9.0-android]

Could update the void TapInsideFrame.OnTapGestureRecognizerTapped(object sender, EventArgs args) sender, to object?.

@jsuarezruiz
Copy link
Contributor

  Warning: /Users/runner/work/maui-samples/maui-samples/9.0/UserInterface/WorkingWithGestures/TapGesture/TapGesture/TapInsideImage.cs(42,35): warning CS8622: Nullability of reference types in type of parameter 'sender' of 'void TapInsideImage.OnTapGestureRecognizerTapped(object sender, EventArgs args)' doesn't match the target delegate 'EventHandler<TappedEventArgs>' (possibly because of nullability attributes). [/Users/runner/work/maui-samples/maui-samples/9.0/UserInterface/WorkingWithGestures/TapGesture/TapGesture/TapGesture.csproj::TargetFramework=net9.0-ios]
  Warning: /Users/runner/work/maui-samples/maui-samples/9.0/UserInterface/WorkingWithGestures/TapGesture/TapGesture/TapInsideFrame.cs(45,35): warning CS8622: Nullability of reference types in type of parameter 'sender' of 'void TapInsideFrame.OnTapGestureRecognizerTapped(object sender, EventArgs args)' doesn't match the target delegate 'EventHandler<TappedEventArgs>' (possibly because of nullability attributes). [/Users/runner/work/maui-samples/maui-samples/9.0/UserInterface/WorkingWithGestures/TapGesture/TapGesture/TapGesture.csproj::TargetFramework=net9.0-ios]
  Warning: /Users/runner/work/maui-samples/maui-samples/9.0/UserInterface/WorkingWithGestures/TapGesture/TapGesture/TapInsideImage.cs(42,35): warning CS8622: Nullability of reference types in type of parameter 'sender' of 'void TapInsideImage.OnTapGestureRecognizerTapped(object sender, EventArgs args)' doesn't match the target delegate 'EventHandler<TappedEventArgs>' (possibly because of nullability attributes). [/Users/runner/work/maui-samples/maui-samples/9.0/UserInterface/WorkingWithGestures/TapGesture/TapGesture/TapGesture.csproj::TargetFramework=net9.0-android]
  Warning: /Users/runner/work/maui-samples/maui-samples/9.0/UserInterface/WorkingWithGestures/TapGesture/TapGesture/TapInsideFrame.cs(45,35): warning CS8622: Nullability of reference types in type of parameter 'sender' of 'void TapInsideFrame.OnTapGestureRecognizerTapped(object sender, EventArgs args)' doesn't match the target delegate 'EventHandler<TappedEventArgs>' (possibly because of nullability attributes). [/Users/runner/work/maui-samples/maui-samples/9.0/UserInterface/WorkingWithGestures/TapGesture/TapGesture/TapGesture.csproj::TargetFramework=net9.0-android]

Could update the void TapInsideFrame.OnTapGestureRecognizerTapped(object sender, EventArgs args) sender, to object?.

@morning4coffe-dev Could you take a look?

@jsuarezruiz jsuarezruiz closed this Sep 8, 2025
@jsuarezruiz jsuarezruiz reopened this Sep 8, 2025
@jsuarezruiz jsuarezruiz merged commit b50baee into dotnet:main Sep 17, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants