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

Flash cards created with Moodle are broken. #11

Closed
Jarvil opened this issue Nov 6, 2017 · 8 comments
Closed

Flash cards created with Moodle are broken. #11

Jarvil opened this issue Nov 6, 2017 · 8 comments

Comments

@Jarvil
Copy link
Contributor

Jarvil commented Nov 6, 2017

Flash cards created in Moodle are broken.

Hello @fnoks ,

If you create a flash card H5P-activity in Moodle it doesn't load the cards at all and you cannot even change the page (the arrow is still highlighted with glowing outline). It works in version 1.4.0 but not in 1.5.3.

If I do flash cards activity in h5p.org and upload it to moodle it works perfectly but not when I direcly create it with moodle H5P-activity.

Tested with mod_hvp 1.5 and Flashcards (1.5.3) & Moodle 3.1

Still works in mod_hvp 1.3 and Flashcards (1.4.0) & Moodle 3.1

flash1

flash2

flash3

@Jarvil Jarvil changed the title Flash cards created in Moodle are broken. Flash cards created with Moodle are broken. Nov 6, 2017
@fnoks
Copy link
Contributor

fnoks commented Nov 6, 2017

Could you please check the JavaScript console to see if there are any error messages there?

@Jarvil
Copy link
Contributor Author

Jarvil commented Nov 6, 2017

Hello,

Error is;

flashcards.js?ver=1.5.3:288 Uncaught TypeError: H5P.JoubelUI.createTip(...).attr is not a function
at C.H5P.Flashcards.C.addCard (flashcards.js?ver=1.5.3:288)
at C.H5P.Flashcards.C.cardsLoaded (flashcards.js?ver=1.5.3:175)
at HTMLImageElement.load (flashcards.js?ver=1.5.3:66)
at HTMLImageElement.dispatch (jquery.js?ver=2017083100:2)
at HTMLImageElement.v.handle (jquery.js?ver=2017083100:2)

@fnoks
Copy link
Contributor

fnoks commented Nov 14, 2017

Could you check which version H5P.JoubelUI is? Go to the library admin page and search for "Joubel UI"

@Jarvil
Copy link
Contributor Author

Jarvil commented Nov 15, 2017

Hello,

Versions are

Joubel UI (1.2.13)   0 32 68  
Joubel UI (1.3.5)   0 24 19

joubelui

@fnoks
Copy link
Contributor

fnoks commented Nov 16, 2017

It's hard to say what is going on here. Is it possible for you to share the H5P here or send it to me on email?

@Jarvil
Copy link
Contributor Author

Jarvil commented Nov 16, 2017

Hello,

Here it is

jari-flashcards-70.zip

@fnoks
Copy link
Contributor

fnoks commented Nov 21, 2017

Thank you for sharing the H5P with us.

For some reason, there is a <BR> in the first cards tip. You can fix it by just opening the editor, and make sure you delete anything inside the tip wysiyg and the save.

The code should have handled this. I am looking into a fix.

@fnoks
Copy link
Contributor

fnoks commented Nov 21, 2017

The fix for this issue is released in Flashcards 1.5.4

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

No branches or pull requests

2 participants