Skip to content
This repository was archived by the owner on May 1, 2024. It is now read-only.

Commit 5a68854

Browse files
Co-authored-by: Brandon Minnick <[email protected]>
1 parent 08dd9bb commit 5a68854

File tree

2 files changed

+12
-4
lines changed

2 files changed

+12
-4
lines changed

src/CommunityToolkit/Xamarin.CommunityToolkit/Effects/Touch/GestureManager.shared.cs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -198,7 +198,6 @@ internal void HandleLongPress(TouchEffect sender)
198198
var longPressAction = new Action(() =>
199199
{
200200
sender.HandleUserInteraction(TouchInteractionStatus.Completed);
201-
sender.LongPressCommand?.Execute(sender.LongPressCommandParameter ?? sender.CommandParameter);
202201
sender.RaiseLongPressCompleted();
203202
});
204203

@@ -229,7 +228,6 @@ internal void OnTapped(TouchEffect sender)
229228
if (sender.Element is IButtonController button)
230229
button.SendClicked();
231230

232-
sender.Command?.Execute(sender.CommandParameter);
233231
sender.RaiseCompleted();
234232
}
235233

src/CommunityToolkit/Xamarin.CommunityToolkit/Effects/Touch/TouchEffect.shared.cs

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1160,10 +1160,20 @@ internal void RaiseHoverStatusChanged()
11601160
=> weakEventManager.RaiseEvent(Element, new HoverStatusChangedEventArgs(HoverStatus), nameof(HoverStatusChanged));
11611161

11621162
internal void RaiseCompleted()
1163-
=> weakEventManager.RaiseEvent(Element, new TouchCompletedEventArgs(CommandParameter), nameof(Completed));
1163+
{
1164+
var element = Element;
1165+
var parameter = CommandParameter;
1166+
Command?.Execute(parameter);
1167+
weakEventManager.RaiseEvent(element, new TouchCompletedEventArgs(parameter), nameof(Completed));
1168+
}
11641169

11651170
internal void RaiseLongPressCompleted()
1166-
=> weakEventManager.RaiseEvent(Element, new LongPressCompletedEventArgs(LongPressCommandParameter ?? CommandParameter), nameof(LongPressCompleted));
1171+
{
1172+
var element = Element;
1173+
var parameter = LongPressCommandParameter ?? CommandParameter;
1174+
LongPressCommand?.Execute(parameter);
1175+
weakEventManager.RaiseEvent(element, new LongPressCompletedEventArgs(parameter), nameof(LongPressCompleted));
1176+
}
11671177

11681178
internal void ForceUpdateState(bool animated = true)
11691179
{

0 commit comments

Comments
 (0)