When you view the site in a different language than English, the icon on the first toolbar link misses.

This is caused by the translation of the classname (toolbar-icon-inhoud instead of toolbar-icon-content, in Dutch).

See screenshot, I added the classnames below. For some strange reason, only Content is translated, where the other items work fine.

Toolbar item Content misses icon

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

BarisW’s picture

Component: toolbar.module » system.module
Status: Active » Needs review
FileSize
1.03 KB

I think I found the bug. The Standard Profile inserts menu links (node/add, admin/content and ), with a t() function added.

Example:

<?php
$menu_link = entity_create('menu_link', array(
    'link_path' => 'admin/content',
    'link_title' => t('All content'),
    'weight' => -19,
  ));
?>

The link titles go through t() later in the process, so we should not use the t() function when adding menu links. The patch removes the t() wrappers in the standard.profile.

jhedstrom’s picture

Status: Needs review » Needs work
Issue tags: +Needs reroll
rpayanm’s picture

Status: Needs work » Needs review
Issue tags: -Needs reroll
FileSize
767 bytes

mgifford queued 3: 2150331-3.patch for re-testing.

NikitaJain’s picture

FileSize
78.46 KB

Not able to reproduce the issue.
Steps:
1. Selected Dutch language at the time of installation of Drupal 8 set-up.
2. Icon is appearing for content link toolbar when viewing in 'Dutch' language.
Screenshot attached.

jhedstrom’s picture

Status: Needs review » Closed (cannot reproduce)

Thanks for confirming @NikitaJain! This appears to have been fixed elsewhere, closing out.