Closed (won't fix)
Project:
Advanced Menu
Version:
4.7.x-1.x-dev
Component:
Miscellaneous
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
7 Dec 2006 at 09:53 UTC
Updated:
14 Jun 2011 at 03:35 UTC
Jump to comment: Most recent
Comments
Comment #1
kkaefer commentedThe problem is that adding a
targetattribute requires really heavy patching to thetheme_menu_item_*functions. There is currently no clean way to store the state of such a menu item. It would require an additional table and lots of additional queries. And that would result in a severe performance hog.While I appreciate your feature request and definitely see the use of it (even though I don’t like when websites open new windows; I use tabbed browsing), I can’t implement this in
advanced_menu.module. This module is really only about exposing the already available settings in the database. The handling for these features (Show in tree, Show in breadcrumb, Hide without children) is already in Drupal, there is just no interface for it.Comment #2
keesje commentedI understand, maybe this should be issued elsewhere (core?.)
I like to use the target property for opening PDF files from the menu (realy, this is wanted occasionaly.) Modern browsers open PDF's in the main window by default.
Comment #3
Anonymous (not verified) commented