On reviewing the code, I ran CodeSniffer and it suggested some changes. I made (most of) the suggested changes and rolled them into the patch that I will put into the next post.

CommentFileSizeAuthor
#1 views_ajax_fade-codesniffer_changes-1895872-02.patch3.87 KBdeetergp

Comments

deetergp’s picture

deetergp commented
Assigned: Unassigned » deetergp
Status: Active » Needs review
StatusFileSize
new views_ajax_fade-codesniffer_changes-1895872-02.patch3.87 KB

And now that I've got an issue number, I can add the patch.

tlattimore’s picture

tlattimore commented
Assigned: deetergp » Unassigned
Status: Needs review » Fixed

Committed the changes from #1 here.

Status: Fixed » Closed (fixed)

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