Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a step by step basic usage guide #167

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kill9zombie
Copy link

Hopefully this should resolve #150

Adds a basic 'direct use' guide for people not so familiar with front end web development.

However, there is a caveat. At the moment examples downloaded from https://h5p.org/content-types-and-applications
don't seem to have the libraries inside the .h5p file!

The H5P docs don't seem to have changed either, so it looks like the libraries should still be there:
https://h5p.org/documentation/developers/h5p-specification

So as it stands, the guide won't work as is with the h5p.org examples. I think we have three options at the moment:

  1. Wait until H5P Group fix their examples
  2. Wait until H5P Group update their docs (maybe they're moving towards a central download of all content types?)
  3. Add a note in the guide to explain what's going on at the moment

Feel free to edit it. Thank you so much for writing this H5P player!

Phil Helliwell and others added 2 commits February 2, 2025 13:46
Adds a basic usage guide for people not so familiar with
front end web development.

Includes a follow on guide to adding a loading indicator.
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.

Request for detailed instructions
1 participant