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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

droplet’s picture

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

nod_
Primary language French
Location Lille
CreditAttribution: nod_ 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
CreditAttribution: sun commented

Thank you!

Dries’s picture

Dries CreditAttribution: 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
CreditAttribution: nod_ commented
Status: Needs work » Needs review
FileSize
core-js-menu-indent-1420048-5.patch3.34 KB
sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun 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
CreditAttribution: nod_ commented
FileSize
core-js-menu-indent-1420048-7-D7.patch3.3 KB
nod_’s picture

nod_
Primary language French
Location Lille
CreditAttribution: nod_ commented
Dries’s picture

Dries CreditAttribution: 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.