Enable images in question content

[h5p]

Enable images in question content

[nadavkav]

wow! I was not aware of this.
Added my +1 to this feature.

On 9 Aug 2016 11:44, "Pål Jørgensen" [email protected] wrote:

Are you guys referring to these https://tracker.moodle.org/browse/MDL-44078
hooks?

@nadavkav https://github.com/nadavkav: No, we are talking about this one:
h5p/moodle-mod_hvp#26
h5p/moodle-mod_hvp#26

—
You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub
#11 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOHAH5rfi1OtsnU8zV3LjrSpNzZ0efxks5qeD4IgaJpZM4JOxgz
.

[icc]

In an H5P context, the WYSIWYG is mostly intended for textual content and not as much for structure and visuals. Allowing images in the text would make the content more unpredictable and it will become more difficult to make changes to the content types without breaking anything. Ideally, the images that are needed should be added as separate fields.

Also, UX wise, I think it's preferable to create familiar patterns for the users.

I don't think this is something that will be including per default in H5P, but you are more than welcome to create 3rd party plugins that modifies the semantics and add support for it.
I would not recommend modifying the content types directly as this would create 'corrupt' versions of the content types. In such case, you should fork the content type and give it a different name.

[nadavkav]

It seems to me that content creators (and their QA team) should be responsible for checking out the outcome of inserting images into content on different devices. making the UI more complicated by adding new input fields for images and their properties will render the editing UI too complex.

Anyways, I talked about this with @leac (offline) and I still think she and the pedagogical content creators are right. So I guess we will fork it, as we need this feature on many content contextes.

[falcon-git]

You don't need to fork it to add this. You may use the corresponding hooks to alter the semantics without forking, see this example for Drupal for instance: https://h5p.org/adding-text-editor-buttons

[nadavkav]

Thank you @falcon-git for the link. I read it, but did understand how to implement it in Moodle. I probably need to dive deeper into the H5P code. 😄

[fnoks]

These hooks are not yet available in Moodle. They will be added in a later release.

[nadavkav]

Are you guys referring to these https://tracker.moodle.org/browse/MDL-44078 hooks?

[fnoks]

Are you guys referring to these https://tracker.moodle.org/browse/MDL-44078 hooks?

@nadavkav: No, we are talking about this one: h5p/moodle-mod_hvp#26