this.rtl = $(this.table).css('direction') == 'rtl' ? -1 : 1; // Direction of the table.

   self.rowObject.indent(1 * self.rtl);

self.rtl = 1
1 * 1 = 1

self.rtl = -1
1 * -1 = -1

CommentFileSizeAuthor
#5 tabledrag_5.patch497 byteshaza
#1 tabledrag_1.patch487 byteskevin morse
tabledrag_.patch507 bytesdroplet

Comments

kevin morse’s picture

kevin morse commented
Version: 8.x-dev » 7.x-dev
StatusFileSize
new tabledrag_1.patch487 bytes

Here's the 7.x patch

droplet’s picture

droplet commented
Version: 7.x-dev » 8.x-dev
barraponto’s picture

barraponto
he / him
commented
Title: mirco clean up to tabledrag.js » micro clean up to tabledrag.js
Status: Needs review » Reviewed & tested by the community

Easy.

haza’s picture

haza
Primary language French
commented
Status: Reviewed & tested by the community » Needs work

I think we first need the D8 patch, then backport it to D7.

haza’s picture

haza
Primary language French
commented
StatusFileSize
new tabledrag_5.patch497 bytes

(so, here it is)

haza’s picture

haza
Primary language French
commented
Status: Needs work » Needs review
maidanet’s picture

maidanet commented
Assigned: Unassigned » maidanet
Status: Needs review » Reviewed & tested by the community

Applied the patch, and table drag still works.

maidanet’s picture

maidanet commented

#5: tabledrag_5.patch queued for re-testing.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to 8.x. Moving to 7.x.

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