When there are no components, the core tabedrag.js looks to the faux "add component" row in the component table to find the weight column. This row has a colspan to combine the name and form_key columns. tabledrag.js is smart enough to cope with colspans when finding the weight column (although oddly it is smart enough when hiding them). The solution is to not use a colspan and supply an empty form_key only when there are no components.

CommentFileSizeAuthor
#3 webform-tabledrag-2759623-2.patch1.15 KBDanChadwick
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DanChadwick created an issue. See original summary.

  • DanChadwick committed 2dbf9bc on 7.x-4.x
    Issue #2759623 by DanChadwick: Display required and hide weight when no...
DanChadwick’s picture

Status: Active » Fixed
FileSize
1.15 KB

Committed to 7.x-4.x.

D8 backport needed, if D8 has the same tabledrag.js lack of support for colspan.

DanChadwick’s picture

Version: 7.x-4.x-dev » 8.x-4.x-dev
Status: Fixed » Patch (to be ported)
DanChadwick’s picture

Category: Bug report » Task
fenstrat’s picture

Version: 8.x-4.x-dev » 7.x-4.x-dev
Category: Task » Bug report
Status: Patch (to be ported) » Fixed

Closing to clear out the old Webform 8.x-4.x branch. See #2827845: [roadmap] YAML Form 8.x-1.x to Webform 8.x-5.x.

  • DanChadwick committed 2dbf9bc on 8.x-5.x
    Issue #2759623 by DanChadwick: Display required and hide weight when no...

Status: Fixed » Closed (fixed)

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