Closed (fixed)
Project:
Title
Version:
7.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
3 Sep 2012 at 11:19 UTC
Updated:
27 Jun 2014 at 23:40 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
gaëlgThe following last three lines should be added for /modules/menu/menu.js to detect the title.
Comment #2
plachPlease, create a patch with the proposed change.
Comment #3
gaëlgAlright. It works as is but it may be a bad way of doing it, so I did not take the time to make a patch. Should it have an isset() check for example? Anyway, the easy fix is to add the class that menu.js is looking for.
Comment #4
plachWell, isset checks never hurt :)
Anyway if this actually fixes the issue it does not look so bad. Did you try it with field groups?
Comment #5
gaëlgNo. All I can say is it works for my needs, with the "default" Title field. I just needed it for use with Entity Translation (the dev version includes an interface to set the menu link title per language in the node form).
But I don't see why it wouldn't work with field groups, as the jquery selector to find the title is simply
var $title = $(this).closest('form').find('.form-item-title input');Comment #6
plachI think Field Groups manipulates the form array hence the line selecting the replacing field wiget might not always work. Not sure about this. We need manual testing.
Comment #7
plachThis won't always be a title or even a label (think of a taxonomy term description). We need to fix this by inheriting the attributes of the legacy field widget (see also #1748008: Replaced fields should inherit the legacy field access).
Comment #8
duellj commentedHow about this patch? Uses the legacy field name instead of hardcoded "title".
Comment #9
nicolas bouteille commentedIt works and no errors. Thanks!
Comment #11
plachCommitted and pushed, thanks.