Accordion editor - tables

Would it be possible to include tables and table formatting  in the Accordion editor (not sure if it's available elsewhere in h5p or not)?

It may help break up long sections of text a bit better

Summary: 
Add tables to accordion editor
Content types: 
Issue Status: 
4
0
Supporter votes Members of the Supporter Network can vote for feature requests. When the supporter network has generated sufficient funding for the top voted feature request it will normally be implemented and released. More about the H5P Supporter Network
tim's picture

Seems like a good feature request :)  Hopefully a developer from the community will pick it up.

It looks like the Accordion Content Type uses "H5P.AdvancedText" for the text area. I created a pull request to be able to use tables. https://github.com/h5p/h5p-advanced-text/pull/6

Let me know what you think.

tim's picture

Super work Nate! Just a quick question before I get this into the review pipeline: have you looked at how this behaves in mobile? Tables are notoriously unresponsive. 

For the table's CSS, I used the Course Presentation's CSS (cp.css). Consistent syling is always good and hopefully mobile has been accounted for too.

I've attached 2 screenshots showing how it looks using Google Chrome Developer Tools. 2 columns and 4 columns work, but much more content and a bad time will be had.

Let me know if there is anything else I can help with.

-Nate

tim's picture

Great stuff. We are going to review it after this release and hopefully we can get it merged in then. 

Hi there, I am trying to find the HTML editor whilst making an Accordian, so that I can remove the underlines of a link.

Could you please tell me where to find this HTML editor.


Regards,

Hannyterm

otacke's picture

Hi Hannyterm!

H5P doesn't support the HTML mode of the CKEditor, but you can use your own CSS. Please have a look at the customization examples at https://h5p.org/documentation/for-developers/visual-changes

Best,

Oliver 

Hi -- was this ever added?  Can we build tables in Accordion now?

Thanks.

 

Was this feature ever added?  Can we build tables in Accordion now?

BV52's picture

Hi Caitlyn,

Unfortunately this has not been added yet.

-BV