[CB-4588] Documentation for notification.confirm should say to pass buttonLabels as Array

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.9.0
• Assignee: Lorin Beer
• Reporter: Peter
• Created: 15/Aug/13 04:32
• Updated: 23/Aug/18 06:46
• Resolved: 04/Sep/13 17:58

Description

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:

console.log("Notification.confirm(string, function, string, string) is deprecated.  Use Notification.confirm(string, function, string, array).");

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

Issue Links

• is duplicated by: CB-3403 notification.confirm docs have not been updated for deprecated signature