Patch based on #1399496: Fixes JS semicolon to following code standard

CommentFileSizeAuthor
#7 core-js-menu-indent-1420048-7-D7.patch3.3 KBnod_
#5 core-js-menu-indent-1420048-5.patch3.34 KBnod_
fix_indent.patch3.33 KBdroplet

Comments

droplet’s picture

droplet commented
Status: Active » Needs review
nod_’s picture

nod_
Primary language French
Location Lille
commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +JavaScript clean-up

trivial patch is trivial, apply and doesn't break anything.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

Thank you!

dries’s picture

dries commented
Status: Reviewed & tested by the community » Needs work

This will need a quick re-roll because I just committed #1399496: Fixes JS semicolon to following code standard .

nod_’s picture

nod_
Primary language French
Location Lille
commented
Status: Needs work » Needs review
StatusFileSize
new core-js-menu-indent-1420048-5.patch3.34 KB
sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Category: task » bug
Priority: Normal » Minor
Status: Needs review » Reviewed & tested by the community
Issue tags: +Needs backport to D7

Thanks!

This could, in fact, be backported, so backporting other fixes doesn't become a total nightmare.

nod_’s picture

nod_
Primary language French
Location Lille
commented
StatusFileSize
new core-js-menu-indent-1420048-7-D7.patch3.3 KB
nod_’s picture

nod_
Primary language French
Location Lille
commented
dries’s picture

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

Committed to 7.x and 8.x. Thanks!

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