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

[Windows] Fix Shell animation and button visual states on Shell nav #26521

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

Conversation

Foda
Copy link
Member

@Foda Foda commented Dec 10, 2024

Description of Change

Fix the Shell navigation to other page visual glitch, and ensure that the released mouse event triggers on buttons that are pressed and then unloaded

Issues Fixed

Fixes #26466

@Foda Foda requested review from jfversluis, jsuarezruiz and a team as code owners December 10, 2024 21:48
@jfversluis jfversluis added the area-navigation NavigationPage label Dec 12, 2024
@@ -9,6 +9,7 @@ public partial class ButtonHandler : ViewHandler<IButton, Button>
{
PointerEventHandler? _pointerPressedHandler;
PointerEventHandler? _pointerReleasedHandler;
bool _isPressed;
Copy link
Member

Choose a reason for hiding this comment

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

Do we need to apply this to ImageButton as well?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, good idea

@PureWeen
Copy link
Member

/rebase

Mike Corsaro added 4 commits December 19, 2024 22:28
* Clear previous page content presenter on unload to prevent brief visual flash
@github-actions github-actions bot force-pushed the foda/ShellAndButton branch from c70b64a to 95e66b7 Compare December 19, 2024 22:28
@PureWeen
Copy link
Member

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

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

Successfully merging this pull request may close these issues.

Regression in GoToAsync behavior on Windows in 9.0.12
4 participants