Needs review
Project:
Menu clone
Version:
7.x-1.0-beta1
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
15 Dec 2011 at 11:38 UTC
Updated:
3 Jul 2012 at 20:46 UTC
Jump to comment: Most recent file
Comments
Comment #1
RKS commentedIn another bug report the user suggested adding check boxes for the menu items you want in the cloned menu. I think this would be your best option to simplify things and make sure it works properly. It doesn't have to be all ajaxy sometimes.
Comment #2
Mołot commentedI agree, it is misleading now. Against Rule of Smallest Surprise.
Just remove that field until it can work properly.
Comment #3
shawn_smiley commentedAttached is a patch that removes both the edit and delete links from the clone menu tab. I removed the edit links for the same reason as mentioned above for the delete links. Namely that they cause the user to edit the original menu link rather than the cloned copy.
Comment #4
shawn_smiley commentedHere is a slightly modified version of the patch in #3. In this version, I've adjusted the instruction text to be a bit more accurate and fixed an error in the form validation related to the maxlength of the menu name.
Comment #5
malc0mn commentedPlease retest with the latest release and let me know if this is fixed now...