Hi, nice module!
I have some remarks regarding the documentation:
- D7 menu paths have been changed; the project page is not updated for this;
- D7-info-file has a new option Configure, which enables to jump to the settings page from the modules-page, i.c. menu page;
- The README.txt does not contain all instructions - you need to go to the project page to see the usage instructions; the message on the menu item link page can easily be looked over. A reference to on the modules page would be nice, too.

Attach patch fixed these issues (except for the project page).

CommentFileSizeAuthor
menu_firstchild_documentation.patch1.44 KBjohnv
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

anrikun’s picture

anrikun CreditAttribution: anrikun commented
Status: Active » Needs review

Thank you. Will review ASAP!

Chris Matthews’s picture

Chris Matthews CreditAttribution: Chris Matthews commented
Version: 7.x-1.0 » 7.x-1.x-dev
Issue summary: View changes
Status: Needs review » Needs work
Issue tags: +Needs rework

The 7 year old patch not surprisingly no longer applies to the latest 7.x-1.x dev snapshot and needs to be reworked.

git apply -v menu_firstchild_documentation.patch
Checking patch menu_firstchild.info...
error: while searching for:
; $Id: menu_firstchild.info,v 1.3 2011/01/21 10:43:00 anrikun Exp $
name = Menu Firstchild
description = Allows to create parent menu items without path that link to their first viewable children.
core = 7.x
dependencies[] = menu
files[] = menu_firstchild.module
files[] = menu_firstchild.install

error: patch failed: menu_firstchild.info:1
error: menu_firstchild.info: patch does not apply
Checking patch README.txt...
Hunk #1 succeeded at 8 (offset -1 lines).