Skip to content

feat: player improvements#82

Open
jeppekroghitk wants to merge 9 commits into
feature/ui-improvementsfrom
feature/player-improvements
Open

feat: player improvements#82
jeppekroghitk wants to merge 9 commits into
feature/ui-improvementsfrom
feature/player-improvements

Conversation

@jeppekroghitk
Copy link
Copy Markdown
Collaborator

@jeppekroghitk jeppekroghitk commented May 12, 2026

Link to ticket

#7515

Description

Based on customer feedback, some changes to the player are needed to make it usable:

  • Move away from using videotool iframe player.
  • Rely on mp3 files only, to provide a HTML-based player with better controls.
  • Save individual media progress.
  • Draggable scrubber and loading animations.

Screenshot of the result

IMG_6243

--

IMG_6244

Checklist

  • My code is covered by test cases.
  • My code passes our test (all our tests).
  • My code passes our static analysis suite.
  • My code passes our continuous integration process.

If your code does not pass all the requirements on the checklist you have to add a comment explaining why this change
should be exempt from the list.

Additional comments or questions

If you have any further comments or questions for the reviewer please add them here.

@jeppekroghitk jeppekroghitk changed the base branch from develop to feature/ui-improvements May 12, 2026 11:32
@jeppekroghitk jeppekroghitk force-pushed the feature/player-improvements branch 7 times, most recently from 132ad9c to ab83047 Compare May 19, 2026 12:10
@jeppekroghitk jeppekroghitk force-pushed the feature/player-improvements branch from ab83047 to 3184927 Compare May 19, 2026 12:17
@jeppekroghitk jeppekroghitk force-pushed the feature/player-improvements branch from 88685ab to 35ce920 Compare May 19, 2026 12:29
@jeppekroghitk
Copy link
Copy Markdown
Collaborator Author

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

🤖 Generated with Claude Code

- If this code review was useful, please react with 👍. Otherwise, react with 👎.

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