<a href="/drupal7/?q=toolbar/toggle&destination=node" title="Hide shortcuts" class="toggle toggle-active">Hide shortcuts</a> </div>

is rendered when the following should be rendered (which is what I get with my patch to be attached as soon as I know the bug number):

<a href="/drupal7/?q=toolbar/toggle&amp;destination=node" title="Hide shortcuts" class="toggle toggle-active">Hide shortcuts</a> </div>

This toggle link is rendered at least when logged in as administrator on the default drupal8 and drupal7 setups (patch applies to both branches).

CommentFileSizeAuthor
#1 drupal-1095498.patch1.1 KBohnobinki
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ohnobinki’s picture

Assigned: ohnobinki » Unassigned
Status: Active » Needs review
FileSize
1.1 KB
David_Rothstein’s picture

Status: Needs review » Reviewed & tested by the community

I confirmed that this patch works as intended, and yeah, using l() here just makes good sense anyway. RTBC.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 7.x and 8.x.

Status: Fixed » Closed (fixed)
Issue tags: -xhtml compliance

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