Stacked tables in Mobile has wrong headers

In looking at this I noticed a few things where the table row is getting processed...

        foreach ($rows as $row) {
          if ($skip_first) {
            $skip_first = FALSE;
            continue;
          }
          $counter = 0;
          $tbodyTags = $xpath->query('//tbody//td|//th', $row);
          foreach ($tbodyTags as $tag) {
            if ($tag->nodeName === 'th') {
              $tag->setAttribute('scope', 'row');
            }
            else {
              $data_label = $table_headers[$counter];
              $tag->setAttribute('data-label', $data_label);
            }
          }
        }

in the foreach loop internal to the row the $counter variable is never incremented.

I also noticed that the xpath query $tbodyTags = $xpath->query('//tbody//td|//th', $row); returns a list of 25 items... so it was looping through not only the items in the current row, but it was also looping through all of the cells in the table for each row.

I modified this to instead just grab the

$row->childNodes</a> of the current row and this resolved the issue... the same section of code work if you have this instead
<code>
        foreach ($rows as $row) {
          if ($skip_first) {
            $skip_first = FALSE;
            continue;
          }
          $counter = 0;
          foreach ($row->childNodes AS $tag) {
            switch ($tag->nodeName) {
              case 'th':
                $tag->setAttribute('scope', 'row');
              case 'td':
                $data_label = $table_headers[$counter];
                $tag->setAttribute('data-label', $data_label);
                $counter++;
                break;
              default:
                error_log($tag->nodeName);
            }
          }
        }

here is my patch

found a couple of small issues with my patch

Confirmed the issue and that your MR 62 fixed it thanks!

Automatically closed - issue fixed for 2 weeks with no activity.