Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Sreenivas Bttv created an issue. See original summary.

Sreenivas Bttv’s picture

Assigned: Unassigned » Sreenivas Bttv
Sreenivas Bttv’s picture

FileSize
0 bytes

Please review patch comments_theming-3084567.patch

Sreenivas Bttv’s picture

Assigned: Sreenivas Bttv » Unassigned
Status: Active » Needs review
Sreenivas Bttv’s picture

#3 patch empty. so again adding a new patch. Please review -> comments_theming-3084567-2.patch

Sreenivas Bttv’s picture

FileSize
10.26 KB

#5 patch failing because of conflicts and white space issues. Also noticed in node. So updated new patch.
So please review patch -> comments_theming-3084567-3.patch

sd9121’s picture

Assigned: Unassigned » sd9121
sd9121’s picture

@Sreenivas Bttv , I reviewed your #6 patch and it's LGTM.

sd9121’s picture

Assigned: sd9121 » Unassigned
Status: Needs review » Fixed

  • Sreenivas Bttv committed 571d9c9 on 8.x-1.x
    Issue #3084567 by Sreenivas Bttv, sd9121: Comments View theming
    
Sreenivas Bttv’s picture

Status: Fixed » Closed (fixed)

@sd9121, Thanks for reviewing the patch.
Patch committed and pushed to 8.x-1.x branch.