brianchance / MvvmCross-UserInteraction

MvvmCross plugin for interacting with the user from a view model
MIT License
43 stars 44 forks source link

Added ConfirmThreeButtons #7

Closed geirsagberg closed 10 years ago

geirsagberg commented 10 years ago

Added methods to Android and iOS for showing a confirm dialog with three buttons: positive, negative and neutral.

brianchance commented 10 years ago

Sorry for the long reply, playing devils advocate, would a Nullable be better than the ConfirmResponse. If we keep the ConfirmResponse, it should probably be named ConfirmThreeButtonResponse.

geirsagberg commented 10 years ago

I'm not a big fan of using Nullables semantically, I find enums are more readable and clearer in their intent. I can change it to ConfirmThreeButtonsResponse though.

geirsagberg commented 10 years ago

Made the change. Sorry about the Resource.designer.cs and the different tab spacing.. haven't found a good way to keep original formatting when using ReSharper...

brianchance commented 10 years ago

Thanks, should be out on nuget.

geirsagberg commented 10 years ago

Cool, thanks for the fast update :)