Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #11431 from wojtekmaj/secondary-toolbar-const

Browse Source 
Use const in secondary_toolbar.js
This commit is contained in:
Tim van der Meij 2019-12-24 00:03:58 +01:00 committed by GitHub
commit a6188a5c98
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
web/secondary_toolbar.js
View File

@ -12,6 +12,7 @@
* See the License for the specific language governing permissions and
* limitations under the License.
*/
/* eslint no-var: error, prefer-const: error */
import { SCROLLBAR_PADDING, ScrollMode, SpreadMode } from './ui_utils';
import { CursorTool } from './pdf_cursor_tools';
@ -172,8 +173,8 @@ class SecondaryToolbar {
for (const { element, eventName, close, eventDetails, } of this.buttons) {
element.addEventListener('click', (evt) => {
if (eventName !== null) {
let details = { source: this, };
for (let property in eventDetails) {
const details = { source: this, };
for (const property in eventDetails) {
details[property] = eventDetails[property];
}
this.eventBus.dispatch(eventName, details);