Description

Anchor menu items is a Drupal module that provides anchor menu items.
Drupal core doesn't support anchors by default, this module makes it possible to add anchors in menu items

Integrates with scroll_to_destination_anchors

Sandbox + git

https://drupal.org/sandbox/robiningelbrecht/2205641
git clone --branch 7.x-1.x http://git.drupal.org/sandbox/robin.ingelbrecht/2205641.git anchor_menu_items

Drupal version

7.x

Reviews of other projects

https://drupal.org/comment/8523539#comment-8523539
https://drupal.org/comment/8523709#comment-8523709

Ran project trough pareview

Comments

robin.ingelbrecht’s picture

Issue summary: View changes
robin.ingelbrecht’s picture

Status: Active » Needs review
contactvishaljain’s picture

Status: Needs review » Needs work

Hi robin.ingelbrecht,

PAReview is reporting issue on the code. Please see http://pareview.sh/pareview/httpgitdrupalorgsandboxrobiningelbrecht22056....

robin.ingelbrecht’s picture

robin.ingelbrecht’s picture

Issue summary: View changes
Status: Needs work » Needs review
robin.ingelbrecht’s picture

Issue summary: View changes
robin.ingelbrecht’s picture

Issue summary: View changes
robin.ingelbrecht’s picture

Issue summary: View changes
robin.ingelbrecht’s picture

PA robot’s picture

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.

gobinathm’s picture

@vishal_yug, please perform full manual review.

@robin.ingelbrecht suggestion on the application request.

1. Please update your project description to include an non-maintainer version of GIT repo clone command.
2. Project applications with Review bonus takes priority. So please complete manual reviews & include those url's here. Once done, please include appropriate tags in the application.

Note: the above comments are not based on project review. Those are simple suggestion on project application.

robin.ingelbrecht’s picture

Issue summary: View changes

Updated project description to include an non-maintainer version of GIT repo clone command.

mehul.shah’s picture

Hi Robin,

1. In validation anchor_menu_items_validate
You are validation only 1st part of link path instead you have to check the whole path after .

2. Condition for 2 anchor tags is not handled?
Ex. //abc

3. Condition for anchor tag at other position i.e. except 1st position is not handled
Ex. abc/ OR abc/xyz/

robin.ingelbrecht’s picture

Status: Needs review » Needs work
robin.ingelbrecht’s picture

Status: Needs work » Needs review

Hi mehul.shah,

1. In validation anchor_menu_items_validate
I validate everything except for the last element because that is the anchor element. For example: admin/structure/blocks#anchor => admin/structure/blocks gets validated

2. Condition for 2 anchor tags is not handled?
You can't use 2 anchor tags on one url?

3. Condition for anchor tag at other position i.e. except 1st position is not handled
The module expects <anchor>/admin/structure/blocks/firstblock as input and it will format the link as admin/structure/block#firstblock.
So the last element is always the anchor element, the first is a placeholder so that drupal knows it's an anchor link and everything between is the link itself.

robin.ingelbrecht’s picture

Issue summary: View changes

updated description :
Integrates with scroll_to_destination_anchors

simon.westyn’s picture

Status: Needs review » Reviewed & tested by the community

No objections, everything looks good and clean!
Works perfectly!

pingwin4eg’s picture

Status: Reviewed & tested by the community » Needs work

@simon.westyn
You may not set status to RTBC without reviewing the application.

@robin.ingelbrecht
Can you explain please (on the project's page) what is so extra your module does that Drupal core does not provide by default? "#" sign can be used normally in menu items.

slydevil’s picture

Robin, I hate to tell you this, but there is another module that will allow you to do this very thing.

1.2 Ensure your project is not a duplication

Check out the Menu Token module: https://www.drupal.org/project/menu_token

Not only does it allow you to add anchors, but it also allows query strings and tokens.

pingwin4eg’s picture

Status: Needs work » Postponed (maintainer needs more info)

Postponed, according to last 2 comments. Please, add a description of how your project differs from others.

robin.ingelbrecht’s picture

This project is an duplicate of https://www.drupal.org/project/menu_token.
You can remove it...

PA robot’s picture

Status: Postponed (maintainer needs more info) » Closed (duplicate)
Multiple Applications
It appears that there have been multiple project applications opened under your username:

Project 1: https://www.drupal.org/node/2346321

Project 2: https://www.drupal.org/node/2205693

As successful completion of the project application process results in the applicant being granted the 'Create Full Projects' permission, there is no need to take multiple applications through the process. Once the first application has been successfully approved, then the applicant can promote other projects without review. Because of this, posting multiple applications is not necessary, and results in additional workload for reviewers ... which in turn results in longer wait times for everyone in the queue. With this in mind, your secondary applications have been marked as 'closed(duplicate)', with only one application left open (chosen at random).

If you prefer that we proceed through this review process with a different application than the one which was left open, then feel free to close the 'open' application as a duplicate, and re-open one of the project applications which had been closed.

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

avpaderno’s picture