When using version 7.x-1.0-beta1 I get the following:
Beta

When using version 7.x-1.x-dev (2014-Nov-20):
Beta

Why was the markup changed in the commits after that beta? Adding labels and replacing the header breaks the original structure.

CommentFileSizeAuthor
no_table_drag-dev.png9.97 KBmalberts
no_table_drag-beta.png5.72 KBmalberts

Comments

minoroffense’s picture

minoroffense commented

We changed the markup to make the table more compliant with WCAG 2.0 and AODA requirements. There are a few more changes coming along those lines.

We're thinking of adding options to tweak certain settings though. Allow users to set how the tables should render which may fix your issue with the markup.

malberts’s picture

malberts commented

Thank you. That makes sense, but my assumption with this module is that its purpose is to disable table dragging. If it also changes the markup, especially away from the Drupal defaults, then it is a bit confusing. If my intention was only to remove the dragging, I would have to replace the entire no_table_drag_theme_field_multiple_value_form() in the registry again, which defeats the purpose of using this module in the first place as the rest of the code is minor compared to that.

If there's a setting for something like "Drupal default" vs "Accessible" markup then I think it would be fine.

I am using the beta version for now, so this isn't an urgent issue for me.