Skip to content

Commit 5bdaa11

Browse files
luizhf42gustavosbarreto
authored andcommitted
fix(ui): fix player's time handlers
- getCurrentTime and getDuration now return promises, needing to be awaited
1 parent bca74e7 commit 5bdaa11

File tree

1 file changed

+12
-12
lines changed

1 file changed

+12
-12
lines changed

ui/src/components/Sessions/Player.vue

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -118,20 +118,20 @@ const currentSpeed = ref(1);
118118
119119
const changeFocusToPlayer = () => { playerWrapper.value?.focus(); };
120120
121-
const getCurrentTime = () => { currentTime.value = player.value.getCurrentTime(); };
121+
const getCurrentTime = async () => { currentTime.value = await player.value.getCurrentTime(); };
122122
123-
const getDuration = () => { duration.value = player.value.getDuration(); };
123+
const getDuration = async () => { duration.value = await player.value.getDuration(); };
124124
125-
const changePlaybackTime = (value: number) => {
125+
const changePlaybackTime = async (value: number) => {
126126
player.value.seek(value);
127-
getCurrentTime();
127+
await getCurrentTime();
128128
};
129129
130130
const clearCurrentTimeUpdater = () => {
131131
clearInterval(timeUpdaterId.value);
132132
};
133133
134-
const startCurrentTimeUpdater = () => {
134+
const startCurrentTimeUpdater = async () => {
135135
clearCurrentTimeUpdater(); // clear to prevent multiple intervals when replaying
136136
timeUpdaterId.value = window.setInterval(getCurrentTime, 100);
137137
};
@@ -163,11 +163,11 @@ const createPlayer = (startAt = 0) => {
163163
};
164164
165165
const setPlayerEventListeners = () => {
166-
player.value.addEventListener("playing", () => {
166+
player.value.addEventListener("playing", async () => {
167167
sessionEnded.value = false;
168-
getCurrentTime();
168+
await getCurrentTime();
169169
startCurrentTimeUpdater();
170-
getDuration();
170+
await getDuration();
171171
});
172172
173173
player.value.addEventListener("ended", () => {
@@ -176,13 +176,13 @@ const setPlayerEventListeners = () => {
176176
clearCurrentTimeUpdater();
177177
});
178178
179-
useEventListener(containerDiv.value, "keydown", (event: KeyboardEvent) => {
180-
getCurrentTime();
179+
useEventListener(containerDiv.value, "keydown", async (event: KeyboardEvent) => {
180+
await getCurrentTime();
181181
if (event.key === "Escape") emit("close");
182182
});
183183
184-
useEventListener(containerDiv.value, "keyup", () => {
185-
getCurrentTime();
184+
useEventListener(containerDiv.value, "keyup", async () => {
185+
await getCurrentTime();
186186
});
187187
};
188188

0 commit comments

Comments
 (0)