cordova-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (JIRA)" <>
Subject [jira] [Commented] (CB-7639) Notification.confirm() fails on windows due to multiple issues
Date Thu, 13 Nov 2014 06:53:34 GMT


ASF subversion and git services commented on CB-7639:

Commit cd9c3bb5b1d8a047aeb8615f756cc19dc43eda4e in cordova-plugin-dialogs's branch refs/heads/master
from [~vladimir.kotikov]
[;h=cd9c3bb ]

CB-7639 Revert back isAlertShowing flag in case of exception to prevent queuing of future

> Notification.confirm() fails on windows due to multiple issues
> --------------------------------------------------------------
>                 Key: CB-7639
>                 URL:
>             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

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message