HomePhabricator

Use `window.confirm` instead of `confirm`

Description

Use window.confirm instead of confirm

Summary:
This silences the following JSHint warning:

>>> Lint for webroot/rsrc/js/application/uiexample/notification-example.js:

   Warning  (W117) JSHintW117
    'confirm' is not defined.

              39           notification.listen(
              40             'activate',
              41             function() {
    >>>       42               if (!confirm('Close notification?')) {
              43                 JX.Stratcom.context().kill();
              44               }
              45             });

Test Plan: arc lint

Reviewers: epriestley, Blessed Reviewers

Reviewed By: epriestley, Blessed Reviewers

Subscribers: Korvin, epriestley

Differential Revision: https://secure.phabricator.com/D11065

Details

Provenance
joshuaspenceAuthored on
epriestleyCommitted on Dec 30 2014, 10:55 AM
epriestleyPushed on Dec 30 2014, 10:55 AM
Reviewer
Blessed Reviewers
Differential Revision
D11065: Use `window.confirm` instead of `confirm`
Parents
rP558e1ee3b8d2: Define JavaScript functions before they are used
Branches
Unknown
Tags
Unknown