[
https://issues.apache.org/jira/browse/CB-4588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13758056#comment-13758056
]
ASF subversion and git services commented on CB-4588:
-----------------------------------------------------
Commit 8b1ad9dc31782c1a63f0c305246eb7ed1228eb3c in branch refs/heads/master
from [~lorin]
[ https://git-wip-us.apache.org/repos/asf?p=cordova-docs.git;h=8b1ad9d ]
[CB-4588] added square brackets to default value documentation
> Documentation for notification.confirm should say to pass buttonLabels as
> Array
> -------------------------------------------------------------------------------
>
> Key: CB-4588
> URL: https://issues.apache.org/jira/browse/CB-4588
> Project: Apache Cordova
> Issue Type: Improvement
> Components: Docs
> Affects Versions: 2.9.0
> Reporter: Peter
> Assignee: Lorin Beer
> Priority: Minor
>
> Compare navigator.notification.confirm and navigator.notification.prompt.
> Both APIs allow caller to specifiy *buttonLabels* as 4th parameter, but
> confirm expects a comma separated String and prompt expects an array. The
> parameter passing mechanism should be consistent for these similar APIs.
> It looks like the cordova.js is already handling the scenario, because it
> says:
> {code}
> console.log("Notification.confirm(string, function, string, string) is
> deprecated. Use Notification.confirm(string, function, string, array).");
> {code}
> But the documentation for navigator.confirm is out-of-step with the code; it
> should be updated to say to pass Array, not String.
> Seems related to CB-1933
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
