[
https://issues.apache.org/jira/browse/CB-7639?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14209390#comment-14209390
]
ASF subversion and git services commented on CB-7639:
-----------------------------------------------------
Commit d8d422a4712665a0a63b74aa319769893d6fe17d in cordova-plugin-dialogs's
branch refs/heads/master from [~vladimir.kotikov]
[
https://git-wip-us.apache.org/repos/asf?p=cordova-plugin-dialogs.git;h=d8d422a ]
CB-7639 Handle button labels as array on windows
> Notification.confirm() fails on windows due to multiple issues
> --------------------------------------------------------------
>
> Key: CB-7639
> URL: https://issues.apache.org/jira/browse/CB-7639
> Project: Apache Cordova
> Issue Type: Bug
> Components: Plugin Dialogs
> Reporter: Vladimir Kotikov
>
> There are bunch of issues related to this bug:
> # On WP8.1 it's not possible to show dialog with more than two buttons. Such
> attempt will cause {{WinRTError: The parameter is incorrect}} exception
> # On Windows8/8.1 it is not possible to add more than three buttons to
> MessageDialog instance. It will throw {{JavaScript runtime error: The
> operation attempted to access data outside the valid range}} error.
> These issues should be noted in plugin's docs.
> # After dialog call, that causes any exception such as listed above,
> following attempts to show any dialog will have no effect because of
> windows's implementation logic. We need to handle exceptions inside of plugin
> and always set {{isAlertShowing}} flag to false to prevent queuing of future
> dialogs calls.
> # When confirm dialog is called with buttons array, if one of button's labels
> contains a comma, it will be splitted to two separate labels. For example
> {{["yes","no","maybe,possible"]}} will be converted into
> {{["yes","no","maybe", "possible"]}}
> Also there is issue in manual tests: windows Javascript apps doesn't have
> built-in notification finctions, such as {{alert()}} or {{confirm()}}, so
> manual tests that calls this methods will fail. We need to add check for
> these methods exists to manual tests.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
