Skip to content

fix: miscellaneous bug fixes#1374

Open
DuarteFrugoli wants to merge 1 commit into
DIYgod:masterfrom
DuarteFrugoli:fix/bugs
Open

fix: miscellaneous bug fixes#1374
DuarteFrugoli wants to merge 1 commit into
DIYgod:masterfrom
DuarteFrugoli:fix/bugs

Conversation

@DuarteFrugoli

Copy link
Copy Markdown
  • utils.js: isSafari false-positive on Chrome (was missing Chrome exclusion)
  • timer.js: replace .map() with .forEach() for side-effect-only loops
  • i18n.js: convert ES5 constructor function to class syntax
  • bar.js: get() reads correct CSS property based on direction param
  • thumbnails.js: store thumbnailWidth in resize() so move() uses updated value
  • options.js: shallow clone options/video/danmaku/subtitle to avoid mutating caller's object
  • options.js: fix typo player.infoPanel.triggle() -> toggle()
  • subtitle.js: use getCueAsHTML() DocumentFragment directly via appendChild

- utils.js: isSafari false-positive on Chrome (was missing Chrome exclusion)
- timer.js: replace .map() with .forEach() for side-effect-only loops
- i18n.js: convert ES5 constructor function to class syntax
- bar.js: get() reads correct CSS property based on direction param
- thumbnails.js: store thumbnailWidth in resize() so move() uses updated value
- options.js: shallow clone options/video/danmaku/subtitle to avoid mutating caller's object
- options.js: fix typo player.infoPanel.triggle() -> toggle()
- subtitle.js: use getCueAsHTML() DocumentFragment directly via appendChild
@netlify

netlify Bot commented Jun 3, 2026

Copy link
Copy Markdown

Deploy Preview for dplayerapp canceled.

Name Link
🔨 Latest commit ea72d67
🔍 Latest deploy log https://app.netlify.com/projects/dplayerapp/deploys/6a1fa2be81eb25000ad7ad2a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant