Add RTL support to the H5P Editor

[nadavkav]

We (The RTL folks) missing an RTL/LTR button on the editor's toolbar, everywhere text can be edited.
Example:

(I am referring to Text flow, and not the Alignment)

[nadavkav]

Looking at the underlying CKEditor code, it seem trivial to add the RTL/LTR buttons, even to the "Basic" toolbar layout. as they are very important to us (RTL folks)

Also set config.contentsLangDirection = 'rtl'; when the calculated current language in Moodle for the current user is RTL. when the editor initiates.

Another improvement could be, is to add an rtl class to the h5p iframe and a corresponding RTL stylesheet file. maybe a H5P core library?

[thomasmars]

This must be implemented in core, so closing this issue.

[nadavkav]

Just in case anyone reads this issue...
Here is a link to h5p-moodle-plugin with RTL support:
https://github.com/iucc/h5p-moodle-plugin/commits/v118_rtl_support

[thomasmars]

@nadavkav Thanks, it is a good reference. I see that the library and editor has been swapped out in your branch. That is in line with why I'm closing this issue.
In order to support RTL in H5P it has to be implemented generally in H5P Core and H5P editor, so that it is supported across all H5P integrations (e.g. in h5p/h5p-editor-php-library#14 ). The changes must be made there, not here in the H5P Moodle integration.

[nadavkav]

@thomasmars , I guess I miss understood you and thought you meant "Moodle core" and not H5P core. oups.
I completely agree with you now :-)