| commit | 65be45c9ec5aed9de5be2844c2d1861cb401d10a | |
| author | jamiewalch@chromium.org <jamiewalch@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | |
| Tue, 12 Feb 2013 22:58:07 +0000 (12 22:58 +0000) | ||
| committer | jamiewalch@chromium.org <jamiewalch@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | |
| Tue, 12 Feb 2013 22:58:07 +0000 (12 22:58 +0000) | ||
| tree | 6f98ed7b4fada9e0f159c65ba00eb973f75c2cbe | treesnapshot (tar.gz zip) |
| parent | 0cd50aa38b761f0b4959794664698f19eeeffb23 | commitdiff |
Attach menu dismiss handler to <body>, not <html>.
document.all is not available to Apps v2, so menus were previously not
dismissable. I could use document.body.parentNode, but I can't think of
any good reason to attach the event handler to <html> instead of <body>.
Review URL: https://codereview.chromium.org/12208141
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@182046 0039d316-1c4b-4281-b951-d872f2087c98
document.all is not available to Apps v2, so menus were previously not
dismissable. I could use document.body.parentNode, but I can't think of
any good reason to attach the event handler to <html> instead of <body>.
Review URL: https://codereview.chromium.org/12208141
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@182046 0039d316-1c4b-4281-b951-d872f2087c98
| remoting/webapp/menu_button.js | diffblobblamehistory |