Skip to content

Commit 0429d0d

Browse files
committed
Incorporated reviews and fixed coding standards violations
1 parent 0d6592d commit 0429d0d

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

src/js/listeners.js

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -108,12 +108,12 @@ class Listeners {
108108

109109
case 38:
110110
// Arrow up
111-
this.proxy(event, () => player.increaseVolume(0.1), 'volume')
111+
this.proxy(event, () => player.increaseVolume(0.1), 'volume');
112112
break;
113113

114114
case 40:
115115
// Arrow down
116-
this.proxy(event, () => player.decreaseVolume(0.1), 'volume')
116+
this.proxy(event, () => player.decreaseVolume(0.1), 'volume');
117117
break;
118118

119119
case 77:
@@ -127,23 +127,23 @@ class Listeners {
127127

128128
case 39:
129129
// Arrow forward
130-
this.proxy(event, player.forward, 'fastForward')
130+
this.proxy(event, player.forward, 'fastForward');
131131
break;
132132

133133
case 37:
134134
// Arrow back
135-
this.proxy(event, player.rewind, 'rewind')
135+
this.proxy(event, player.rewind, 'rewind');
136136
break;
137137

138138
case 70:
139139
// F key
140-
this.proxy(event, () => player.fullscreen.toggle(), 'fullscreen')
140+
this.proxy(event, player.fullscreen.toggle, 'fullscreen');
141141
break;
142142

143143
case 67:
144144
// C key
145145
if (!repeat) {
146-
this.proxy(event, () => player.toggleCaptions(), 'captions')
146+
this.proxy(event, player.toggleCaptions, 'captions');
147147
}
148148
break;
149149

@@ -173,7 +173,7 @@ class Listeners {
173173
// Escape is handle natively when in full screen
174174
// So we only need to worry about non native
175175
if (code === 27 && !player.fullscreen.usingNative && player.fullscreen.active) {
176-
this.proxy(event, () => player.fullscreen.toggle(), 'fullscreen')
176+
this.proxy(event, player.fullscreen.toggle, 'fullscreen');
177177
}
178178

179179
// Store last code for next cycle

0 commit comments

Comments
 (0)