Skip to content

Commit

Permalink
fix: time tooltip out of bounds on the right
Browse files Browse the repository at this point in the history
This revives
videojs#7308

The analysis by @PikachEXE
videojs#7248 (comment)
still stands.

However, the alignment issues pointed out in
videojs#7308 (comment)
cannot be reproduced with the current Video.js v8.x and v7.x versions
after the patch is applied.
  • Loading branch information
phloxic committed Oct 26, 2023
1 parent cd2dac4 commit 7f0c253
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/js/control-bar/progress-control/progress-control.js
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ class ProgressControl extends Component {
}

const seekBarEl = seekBar.el();
const seekBarRect = Dom.findPosition(seekBarEl);
const seekBarRect = Dom.getBoundingClientRect(seekBarEl);
let seekBarPoint = Dom.getPointerPosition(seekBarEl, event).x;

// The default skin has a gap on either side of the `SeekBar`. This means
Expand Down

0 comments on commit 7f0c253

Please sign in to comment.