Wrong ordering of args in rowIndentation():
http://api.jquery.com/jquery.each/

Before

After

CommentFileSizeAuthor
tabledrag-indication.patch437 bytesdroplet
after.png13.53 KBdroplet
before.png12.16 KBdroplet
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch, tabledrag-indication.patch, failed testing.

nod_’s picture

Status: Needs work » Reviewed & tested by the community
Issue tags: +Novice, +JavaScript

Ouch, this one is on us. I did wonder why the graph were not pretty anymore but never had the courage to dig into tabledrag code.

Introduced in #1684812: JSHint tabledrag.js.

Patch fixes the issue.

nod_ queued tabledrag-indication.patch for re-testing.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, tabledrag-indication.patch, failed testing.

Status: Needs work » Needs review

droplet queued tabledrag-indication.patch for re-testing.

droplet’s picture

Status: Needs review » Reviewed & tested by the community
alexpott’s picture

Status: Reviewed & tested by the community » Fixed

This issue is a normal bug fix, and doesn't include any disruptive changes, so it is allowed per https://www.drupal.org/core/beta-changes. Committed 796ca9a and pushed to 8.0.x. Thanks!

  • alexpott committed 796ca9a on 8.0.x
    Issue #2499673 by droplet: Add back tabledrag row indication
    

Status: Fixed » Closed (fixed)

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