After I enable Token, I see the "Browse available tokens." hyperlink above the help text.

IMO, this should be moved below the help text and made into a button (since it launches a modal).

This can be done by adding the button button--primary button--small CSS classes to the hyperlink.

The period should also be removed.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mherchel created an issue. See original summary.

mherchel’s picture

Issue summary: View changes
FileSize
56.58 KB

It might be better to not use the button--primary CSS ,and go with button button--small

DamienMcKenna’s picture

Can you move the "make the popup nicer" request to the Token module? Metatag just uses Token to do that bit.

mherchel’s picture

@DamienMcKenna - I'm not sure I know what you're talking about. I didn't put a "make the popup nicer" request (and I just re-checked to check my sanity!).

FWIW, I think the 'popup' modal is nice.

My request is to move the "Browse Available Tokens" hyperlink, and then to add some css classes to it.

DamienMcKenna’s picture

The link is provided by the Token module, any improves to how the link looks should be done there.

That just leaves the token browser link's positioning.

DamienMcKenna’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 5: metatag-n2632028-5.patch, failed testing.

DamienMcKenna’s picture

Status: Needs work » Needs review
DamienMcKenna’s picture

Version: 8.x-1.0-beta3 » 8.x-1.x-dev
DamienMcKenna’s picture

Status: Needs review » Closed (duplicate)
Related issues: +#2648752: Fix token tree browser theme callback