Reviewed & tested by the community
Project:
Comment goodness
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
30 Oct 2012 at 13:25 UTC
Updated:
2 Jul 2022 at 09:25 UTC
Jump to comment: Most recent, Most recent file

Comments
Comment #1
chris.smith commentedFixed number of comments per page in "Comment settings" content type edit form. Needs Review.
Comment #2
nagba commented1) Coding style issues, check your patch in the browser, i can see tabs instead of actual spaces in the indentation. The new function's doxygen also does not follow codying style standards.
2) There is a comment stating "Set "Comments per page" setting to textfield so it allows more gradual control over the number of comments to show." and now you are undoing that. I am not sure that is the right approach.
Comment #3
kaidjohnson commentedI agree with @nagba.
Here's a solution that corrects the javascript selectors in the fieldset summaries. It also includes a small code standards correction.
Comment #4
firewaller commentedPatch #3 works for me
Comment #5
avpadernoComment #6
avpaderno