When the metatags field is used with node entities, it makes sense to have the Metatags fieldsset appear in the "advanced" section, which usually appears on the Right-Hand-Side of the node.

I am not sure if I am the only one who thinks this way, but I will make a patch to see if you guys like it as well.

Comments

jaxxed created an issue. See original summary.

jaxxed’s picture

Status: Active » Needs review
StatusFileSize
new671 bytes

Here is a quick patch to the field widget (firehose) that just adds the entire metatags form elements to the node "advanced" group. If desired, I could add a quick checkbox to the widget configuration, so that the decisions about where to put the fieldset can be made per field.

A note about the implementation:

- this is accomplished by adding '#group' => 'advanced' to the widget form element.
- the node already has an "advanced" group
- if adding the field to an entity that has no advanced group, the setting is ignored (test it with user or something)

* if attaching the widget to an entity that has an advanced group, but into which it doesn't make sense to have the metatags added, then maybe we have a problem.

damienmckenna’s picture

Status: Needs review » Needs work

I didn't do this already on purpose - the Metatag form will have so many meta tags that I thought it would be a bad UX. I'd be willing to accept this if there was a global setting controlling it, but not as-is.

damienmckenna’s picture

Status: Needs work » Needs review
StatusFileSize
new583 bytes

Rerolled as-is.

damienmckenna’s picture

Assigned: jaxxed » Unassigned
Status: Needs review » Fixed
Parent issue: » #2563607: Plan for Metatag 8.x-1.0-beta4 release

Lets just go with this as-is.

  • DamienMcKenna committed 56026cc on 8.x-1.x authored by jaxxed
    Issue #2630068 by jaxxed, DamienMcKenna: Moved the meta tags fields into...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

mvwensen’s picture