A bug to solve for the future maintainer :)
The position of the cursor is kind of wrong calculated.
|
await moveCursor(page, |
|
action.value[0] * currentImage.width, |
|
action.value[1] * currentImage.height |
|
) |
It should be more like this:
https://github.com/bigbluebutton/bbb-playback/blob/ba7e67d8c352d6a5383cc42442178a880db10da7/src/components/presentation/cursor.js#L18-L19
pos_X = current_view_box.x + (action.x * current_view_box.width),
pos_Y = current_view_box.y + (action.y * current_view_box.height),
At least bbb and my bbb-dl uses this calculation, and they look correct.
I also want to point out, that bbb finally published there own script to render the presentation video (without deskshare/webcams) https://github.com/bigbluebutton/bbb-presentation-video. Maybe this can be useful for the future maintainer :D
A bug to solve for the future maintainer :)
The position of the cursor is kind of wrong calculated.
bbb-video-download/src/modules/slides.js
Lines 229 to 232 in 93b894c
It should be more like this:
https://github.com/bigbluebutton/bbb-playback/blob/ba7e67d8c352d6a5383cc42442178a880db10da7/src/components/presentation/cursor.js#L18-L19
At least bbb and my bbb-dl uses this calculation, and they look correct.
I also want to point out, that bbb finally published there own script to render the presentation video (without deskshare/webcams) https://github.com/bigbluebutton/bbb-presentation-video. Maybe this can be useful for the future maintainer :D