panopoly-widgets.css includes the following code:

/* Table Widget - Responsive */
@media only screen and (max-width: 480px) {
  table.tablefield {
    display: none;
  }

  table.mobile-table {
    display: block;
  }

  table {
    margin: 50px;
  }

  td, th {
    padding: 10px;
    text-align: left;
  }
   // etc...
}

As you can see, it includes some styles that apply to ALL tables. It's also a little strange, in that it's hiding the normal table, and replacing it with table.mobile-table but applying styles to all tables (whereas it could just get away with applying to table.mobile-table for doing Table widgets). So, I suspect that was left over from a previous approach at implementing this functionality.

This code originally comes from #1774072: Make Table Widget Responsive which states that the original intent is just to affect Table widgets.

I propose we only apply those styles to table.mobile-table.

Comments

dsnopek’s picture

Status: Active » Needs review
StatusFileSize
new386 bytes

Here's a patch to fix this, but I'm going to hold off on committing it until we make a decision about #2432625: panopoly_widgets makes it hard/impossible to implement responsive tables strategy for the whole site

dsnopek’s picture

Status: Needs review » Closed (won't fix)

Closing, since we decided to just remove this functionality.