Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Hello, i read thats in drupal8 we have responsive tables too. Drupal8 is strong getting into Mobile. So we need to make sure thats in the core wysiwyg Editor we have the tables are responsive.
In my eyes it makes not sense to have a responsive layout/theme but if you use the wyiwyg Editor you don´t have responsive stuff here. In a End-User Eye Drupal has to do this.
Comments
Comment #1
swentel CreditAttribution: swentel commentedThis is really not a critical bug report.
Comment #2
Wim LeersI don't see how we could make that work.
Drupal 8's tables are responsive, but only where using
theme_table()and even then you have to manually specify the priorities for each column:RESPONSIVE_PRIORITY_LOWorRESPONSIVE_PRIORITY_MEDIUM— see https://api.drupal.org/api/drupal/core%21includes%21theme.inc/function/t....The only way we can make "regular" tables (i.e. those not generated by code calling
theme_table(), hence any "content, not code table" entered by a user via the WYSIWYG editor or otherwise) responsive, is by using a filter that adds the necessary JS and CSS to the page and allows the user to specify usingdata-attributes which tables have a higher or lower "responsive priority". You would then probably also want a way to specify that responsive priority through the WYSIWYG editor, without having to edit HTML code manually.This will only benefit people who are embedding huge tables in their content. So I personally think this does not belong in core. It's perfectly plausible to implement this in Drupal 8 (or even 7) contrib, but it does not belong in Drupal core IMO.
Assigning to webchick for feedback, because I know she cares a lot about these kinds of things. I'll let her decide if this issue should be closed or implemented.
This also in no way belongs in the editor.module component — if in any component, it probably belongs in the filter.module component. But I'll leave it here for now.
Comment #3
webchickUnlike #2061377: [drupalImage] Optionally apply image style to images uploaded in CKEditor 5, this feels more edge-casey to me, and something that while important for the entire penultimate mobile experience™ is also that likely comes up very rarely in day-to-day content editing. To make it work, you would need a filter that could read in a
data-attribute likedata-table-priority="low"and do whatever magic is needed on the JS side to make it react in a responsive way.So I think this is a worthy feature request, and would love to see it in D8, but doesn't feel like a bug to me.
Comment #4
jessebeach CreditAttribution: jessebeach commentedIdeally, any approach will produce the same output in terms of HTML structure and classes that
theme_table()outputs. Then,#attaching thedrupal.tableresponsivelibrary will cause a content table to become responsive.Comment #5
webchickI don't think I need to be assigned to this anymore.
Comment #11
sandykadam CreditAttribution: sandykadam commentedHi there,
I'm also facing the similar issue, we are working for one of the telco site and WYSIWYG editor is used extensively, content team is creating lot of content and tables using editor but the tables which are generated are not at all responsive and all our rest of section is responsive. Site is build using D8.5
Please suggest how we can fix this issue, is there any additional library or modules to fix.