Bug 58875 - Back navigation disables Context Action in whole app, if Context Action left open
Summary: Back navigation disables Context Action in whole app, if Context Action left ...
Status: RESOLVED FIXED
Alias: None
Product: Forms
Classification: Xamarin
Component: iOS ()
Version: 2.3.4
Hardware: PC Windows
: Normal blocker
Target Milestone: 2.4.0
Assignee: Samantha Houts [MSFT]
URL:
: 59521 ()
Depends on:
Blocks:
 
Reported: 2017-08-18 09:23 UTC by Dmitijs Jesilevskis
Modified: 2017-09-18 14:41 UTC (History)
5 users (show)

Tags: ac, navigation
Is this bug a regression?: ---
Last known good build:


Attachments
Sample project (299.14 KB, application/x-zip-compressed)
2017-08-18 09:23 UTC, Dmitijs Jesilevskis
Details
2.4.0.269-pre2 (148.20 KB, application/x-zip-compressed)
2017-09-15 14:30 UTC, Dmitijs Jesilevskis
Details


Notice (2018-05-24): bugzilla.xamarin.com is now in read-only mode.

Please join us on Visual Studio Developer Community and in the Xamarin and Mono organizations on GitHub to continue tracking issues. Bugzilla will remain available for reference in read-only mode. We will continue to work on open Bugzilla bugs, copy them to the new locations as needed for follow-up, and add the new items under Related Links.

Our sincere thanks to everyone who has contributed on this bug tracker over the years. Thanks also for your understanding as we make these adjustments and improvements for the future.


Please create a new report on Developer Community or GitHub with your current version information, steps to reproduce, and relevant error messages or log files if you are hitting an issue that looks similar to this resolved bug and you do not yet see a matching new report.

Related Links:
Status:
RESOLVED FIXED

Description Dmitijs Jesilevskis 2017-08-18 09:23:59 UTC
Created attachment 24296 [details]
Sample project

Sample project attached

Pre-requisites:
- Application with Navigation Page style navigation
- Child page contains a ListView, with Context Actions attached to cells

Reproduction steps for Sample project:
1) Tap "Tap me" on the Main Page to open child page
2) Swipe to open Context Actions for any cell, do not close Context Actions
3) Execute Back navigation:
- Use either standard "Back" button of Navigation Page, or
- Swipe from Left to Right starting from the edge of the screen (Back navigation with swipe)
4) Reopen Child Page
5) Try to open Context Actions again.

Actual:
Context Actions could not be opened, until app is killed and restarted.

Expected:
Context Actions should continue to work.

Notes:
Reproduced with:
- iPhone 5C, os 10.3.2 / 10.3.3
- iPad, os 9.3.5
Comment 1 Paul DiPietro [MSFT] 2017-08-18 12:46:26 UTC
This appears fixed as of 2.4.0pre1.
Comment 2 Dmitijs Jesilevskis 2017-09-15 06:35:08 UTC
I've reviewed the issue against 2.4.0.269-pre2.
Now, instead of simply "context menu not opening" behavior, application crashes.
Comment 3 Paul DiPietro [MSFT] 2017-09-15 13:15:04 UTC
I cannot reproduce this crash. The context menu opens and closes after the initial back navigation as expected. Please validate that you have completely cleaned your solution (including bin/obj folders and deleting the app off of your device or emulator) and that your package versions are aligned.
Comment 4 Dmitijs Jesilevskis 2017-09-15 14:30:20 UTC
Created attachment 24776 [details]
2.4.0.269-pre2
Comment 5 Paul DiPietro [MSFT] 2017-09-15 14:32:10 UTC
*** Bug 59521 has been marked as a duplicate of this bug. ***
Comment 6 Dmitijs Jesilevskis 2017-09-15 14:34:49 UTC
I've updated the sample project (just in case).
Also made sure one more time that bin/obj (and packages) are cleaned up.

Here is an adjustment original steps to reproduce the crash mentioned in step 2.
After step 4, Reopening Child page (with the list). I immediately attempt opening the context menu on the list. Then the crash happens.
If I attempt to tap the list, or e.g. select any cell - crash won't happen and the context menu opens.
Comment 7 Paul DiPietro [MSFT] 2017-09-15 15:20:21 UTC
Okay -- I was able to get this to crash on an iPhone 7 with iOS 11, but it should be noted that this is device-only. Emulators do not crash, which is a critical point to note. This will need to be looked into further.
Comment 8 Samantha Houts [MSFT] 2017-09-15 20:07:03 UTC
https://github.com/xamarin/Xamarin.Forms/pull/1144