[D7] Menu link css class [#2122649]

Module provides additional setting for menu links.
Developer or power user can add css style to link, such as css class.

https://drupal.org/node/2114897/
git clone --branch 7.x-1.x http://git.drupal.org/sandbox/impol/2114897.git menu_link_css_class

Comments

Comment #1

PA robot commented 28 October 2013 at 23:23

We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)

Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).

I'm a robot and this is an automated message from Project Applications Scraper.

Comment #2

Michael Hodge Jr commented 28 October 2013 at 23:29

Status:

Needs review

» Needs work

Manual Review

I did a manual review of your module, below are the issues I found.

Duplication: This sounds like a feature that lives in the existing menu_attributes (http://drupal.org/project/menu_attributes) project. Module duplication and fragmentation is a huge problem on drupal.org and we prefer collaboration over competition. Please open an issue in the menu_attributes issue queue to discuss what you need. You should also get in contact with the maintainer(s) to offer your help to move the project forward. If you cannot reach the maintainer(s) please follow the abandoned project process.
If that fails for whatever reason please get back to us and set this back to "needs review".

Code too short: This project is too short to approve you as git vetted user. We are currently discussing how much code we need, but everything with less than 120 lines of code or less than 5 functions cannot be seriously reviewed. However, we can promote this single project manually to a full project for you. I feel that there isn't too much you could possibly do to get over this requirement as your module is straight and to the point (which is a good thing). You could possibly implement hook_perm() and add some inline comments to help get over this threshold.

Project Page: Your project page contains no information. Please take a moment to make your project page follow tips for a great project page.

Usability: I'd suggest changing the weight of the css textfield up on the form so that it is near the path url, and does not look as out of place.