Bug 27119 - WindowsPhone DisplayActionSheet does not await
Summary: WindowsPhone DisplayActionSheet does not await
Status: VERIFIED FIXED
Alias: None
Product: Forms
Classification: Xamarin
Component: Windows ()
Version: 1.4.2
Hardware: PC Windows
: Normal major
Target Milestone: ---
Assignee: Eric Maupin
URL:
: 30377 ()
Depends on:
Blocks:
 
Reported: 2015-02-16 21:59 UTC by Brian
Modified: 2015-07-21 08:05 UTC (History)
9 users (show)

Tags: AC
Is this bug a regression?: ---
Last known good build:


Attachments
Sample Project (271.90 KB, application/zip)
2015-02-17 04:15 UTC, Ram Chandra
Details
Demo app showing bug 27119 (69.15 KB, application/x-gzip)
2015-03-04 19:01 UTC, Paul Hazlett
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:
VERIFIED FIXED

Description Brian 2015-02-16 21:59:24 UTC
On windows phone DisplayActionSheet does not await properly.  The task returns before the user makes a selection.

ToolbarItems.Add(new ToolbarItem("Actions", "actions.png", async () =>
            {
                string _option1 = "Option 1";
                string _option2 = "Option 2";
                
                var _result = await DisplayActionSheet("", "Cancel", "", _option1, _option2);
                
                if (_result == _option1)
                {
                }
                else if (_result == _option2)
                {
                }
            }));
Comment 1 Ram Chandra 2015-02-17 04:15:48 UTC
Created attachment 9865 [details]
Sample Project

I have checked this issue but I am unable to reproduce this issue.

Steps I followed:

1. I have created sample project with the help of bug description.
2. Download the attached sample application
3. Build and deploy on android device
4. Click on toolbar icon
5. Select any option from DisplayActionSheet

I have checked this on windows emulator and observed that it is working fine.

Screencast: http://www.screencast.com/t/BoAGtZz1j

Could you please provide sample application and build info ( Xamarin.VS + Xamarin.Forms) ? So that we can reproduce this issue at our end.

You can get XVS info from here:

Visual Studio => About Microsoft Visual Studio => Copy Info

Environment Info:

Xamarin.Forms.1.3.1.6296
Xamarin.Forms 1.3.3.6323
Xamarin.Forms.1.3.4.6331-pre4

Microsoft Visual Studio Professional 2013
Version 12.0.30723.00 Update 3
Microsoft .NET Framework
Version 4.5.51641
Installed Version: Professional
LightSwitch for Visual Studio 2013   06177-004-0447006-02641
Microsoft LightSwitch for Visual Studio 2013

Team Explorer for Visual Studio 2013   06177-004-0447006-02641
Microsoft Team Explorer for Visual Studio 2013

Visual Basic 2013   06177-004-0447006-02641
Microsoft Visual Basic 2013

Visual C# 2013   06177-004-0447006-02641
Microsoft Visual C# 2013

Visual C++ 2013   06177-004-0447006-02641
Microsoft Visual C++ 2013

Visual F# 2013   06177-004-0447006-02641
Microsoft Visual F# 2013

Visual Studio 2013 Code Analysis Spell Checker   06177-004-0447006-02641
Microsoft® Visual Studio® 2013 Code Analysis Spell Checker

Portions of International CorrectSpell™ spelling correction system © 1993 by Lernout & Hauspie Speech Products N.V. All rights reserved.

The American Heritage® Dictionary of the English Language, Third Edition Copyright © 1992 Houghton Mifflin Company. Electronic version licensed from Lernout & Hauspie Speech Products N.V. All rights reserved.

Windows Phone SDK 8.0 - ENU   06177-004-0447006-02641
Windows Phone SDK 8.0 - ENU

Application Insights Tools for Visual Studio Package   1.0
Application Insights Tools for Visual Studio

ASP.NET and Web Tools   12.3.50717.0
Microsoft Web Developer Tools contains the following components:
Support for creating and opening ASP.NET web projects
Browser Link: A communication channel between Visual Studio and browsers
Editor extensions for HTML, CSS, and JavaScript
Page Inspector: Inspection tool for ASP.NET web projects
Scaffolding: A framework for building and running code generators
Server Explorer extensions for Microsoft Azure Websites
Web publishing: Extensions for publishing ASP.NET web projects to hosting providers, on-premises servers, or Microsoft Azure

ASP.NET Web Frameworks and Tools 2012.2   4.1.21001.0
For additional information, visit http://go.microsoft.com/fwlink/?LinkID=309563

ASP.NET Web Frameworks and Tools 2013   5.2.20703.0
For additional information, visit http://www.asp.net/

Common Azure Tools   1.2
Provides common services for use by Azure Mobile Services and Microsoft Azure Tools.

Microsoft Advertising SDK for Windows Phone   
Microsoft Advertising SDK for Windows Phone
Build 

Microsoft Azure Mobile Services Tools   1.2
Microsoft Azure Mobile Services Tools

NuGet Package Manager   2.8.50926.663
NuGet Package Manager in Visual Studio. For more information about NuGet, visit http://docs.nuget.org/.

Office Developer Tools for Visual Studio 2013 ENU   12.0.30626
Microsoft Office Developer Tools for Visual Studio 2013 ENU

PowerShell Tools   1.2
Provides file classification services using PowerShell

PreEmptive Analytics Visualizer   1.2
Microsoft Visual Studio extension to visualize aggregated summaries from the PreEmptive Analytics product.

SQL Server Data Tools   12.0.30919.1
Microsoft SQL Server Data Tools

Windows Phone 8.1 SDK Integration   1.0
This package integrates the tools for the Windows Phone 8.1 SDK into the menus and controls of Visual Studio.

Workflow Manager Tools 1.0   1.0
This package contains the necessary Visual Studio integration components for Workflow Manager.

Xamarin   3.9.296.0 (d66d164)
Visual Studio extension to enable development for Xamarin.iOS and Xamarin.Android.

Xamarin.Android   4.20.0.34 (49a04b966feb40dfdba49d57ba16249b66d606a6)
Visual Studio plugin to enable development for Xamarin.Android.

Xamarin.iOS   8.6.1.0 (3b3ef438017c7ecf486defa9e01567a5f2b3cb2a)
Visual Studio extension to enable development for Xamarin.iOS.

Xamarin.iOS Unified Migration   1.0
Automated migration for Xamarin iOS Classic projects to Unified
Comment 2 Brian 2015-02-17 09:16:42 UTC
The issue is not with Android.  Check WINDOWS PHONE.
Comment 3 Brian 2015-02-17 09:17:32 UTC
I will submit a sample...
Comment 4 Paul Hazlett 2015-03-04 19:01:00 UTC
Created attachment 10162 [details]
Demo app showing bug 27119

This seems to affect WinPhone only.

To Test:

1) Start the app
2) Click "Show Action Sheet"
3) The label should update with the action sheet result

4) Click "Replace App.MainPage"
5) Click "Show Action Sheet"
6) Note the label text stating that the action sheet was not awaited

The same is true if you click "Navigation.PushAsync".
Comment 5 Paul Hazlett 2015-03-08 05:52:26 UTC
I vote for a priority bump on this issue because action sheets are used extensively in so many applications. I just don't see how it's possible to release a WinPhone app using Xamarin.Forms while this issue exists.
Comment 6 Adam 2015-04-21 13:22:06 UTC
Priority bump please. When you change the MainPage, the await no longer works.
Comment 7 Elvin 2015-05-16 04:56:25 UTC
Xamarin, Priority bump please
Comment 8 Brian 2015-05-26 09:23:41 UTC
Bug still exists.
Comment 9 Adam 2015-05-26 09:56:04 UTC
Just to make it crystal clear and to hopefully get this on the CONF list, the issue only occurs once the MainPage has been changed after it has been initially set.

For example you initially set

Application.MainPage = new MyNavPage();

in your App.cs. Then later in code you change it again to

Application.MainPage = new MyOtherNavPage();

After this point, DisplayAlert and DisplayActionSheet both no longer wait for a user selection or action.

e.g.

await DisplayAlert("title", "message", "accept", "cancel");

will not wait for any user input. It will show the Alert or ActionSheet correctly but the code will move on to the next line without waiting for the user to press accept or cancel.

This only affects Windows Phone and does not affect iOS or Android.
Comment 10 Eric Maupin 2015-05-26 16:41:41 UTC
*** Bug 30377 has been marked as a duplicate of this bug. ***
Comment 11 Erik Renaud 2015-06-19 11:01:15 UTC
I have this bug also ! sam scenario
DisplayActionSheet doesnt' await if you invoke it after changing the mainpage.
Comment 12 Erik Renaud 2015-06-19 11:01:35 UTC
still present in 1.4.3-pre3
Comment 13 Jason Smith [MSFT] 2015-06-19 15:45:41 UTC
Should be fixed in 1.4.4-pre1
Comment 14 Parmendra Kumar 2015-07-21 08:05:37 UTC
I have checked this issue with Xamarin.Forms.1.4.4.6379-pre3 and its working fine at my end.

Hence closing this issue.