User:Santhosh.thottingal/i18nreview/ArticleFeedbackv5

Unused messages

 * The special page ArticleFeedbackv5 uses jquery.articleFeedbackv5.special module. All the messages used in that js module should be registered as messages in resource loader to make it available at client side  is not registered as a message in ArticleFeedbackv5.php
 * For jquery.articleFeedbackv5.special module,several messages are regsitered, but I don't see the following messages used a client side:
 * The above comment is valid for jquery.articleFeedbackv5 module. Not all messages registered for the module is not used at client side. remove unwanted messages from module definition.
 * Remove all unused messages from i18n.php file. The following messages are found only in i18n.php and not used anywhere. It is a waste of time for translators
 * articlefeedbackv5-error-abuse-linktext
 * articlefeedbackv5-error-abuse-link
 * articlefeedbackv5-special-sort-age
 * articlefeedbackv5-special-sort-helpful
 * articlefeedbackv5-special-sort-rating
 * articlefeedbackv5-form-optionid
 * articlefeedbackv5-form-header
 * articlefeedbackv5-special-title
 * articleFeedbackv5-table-caption-dailyhighsandlows
 * articleFeedbackv5-table-caption-dailyhighs
 * articleFeedbackv5-table-caption-dailylows
 * articleFeedbackv5-table-caption-weeklymostchanged
 * articleFeedbackv5-table-caption-recentlows
 * articleFeedbackv5-table-heading-page
 * articleFeedbackv5-table-heading-average
 * articlefeedbackv5-table-noratings
 * articleFeedbackv5-copy-above-highlow-tables
 * articlefeedbackv5-dashboard-bottom
 * articlefeedbackv5-privacyurl

Message documentation
It is mandatory to have documention for all the interface messages. See I18n.
 * Messages like All ($1), Unhelpful ($1) should have documentation about what is $1, because translators should know what is going to be be there at placeholders for a meaningful message in their language.

General Comments

 * Consider changing the way modules are registered. ResourceLoaderRegisterModules hook is discouraged and alternate methods are suggested in Manual:Hooks/ResourceLoaderRegisterModules