Gin Toolbar TMGMT Icon Issue

Currently, the Gin Toolbar successfully displays the TMGMT translation icon. However, because it is displayed as a background image, its hover and active states are out-of-sync with the expected behavior for the toolbar.

Patch to put the SVG as a `mask-image` forthcoming.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kmonty created an issue. See original summary.

kmonty’s picture

saschaeggi’s picture

Status: Active » Needs review

Thanks @kmonty
The icon is now included. I've used your patch but also took the time to refactor the SVG icon handling. It's now using an SVG sprite, which makes the maintenance easier and also adding new icons to Gin.
Long story short, you can checkout the latest 3.x-dev for testing.

rkoller’s picture

Status: Needs review » Needs work
FileSize
67.52 KB

I've updated to the latest 3.x-dev yesterday night... already wanted to open up a new issue today but I've figured it might be suitable for this issue as well. With the latest dev version I get an error in the devtools console:

[Error] Failed to load resource: the server responded with a status of 404 () (sprite.svg, line 0) https://d9gin.ddev.site/themes/custom/gin/dist/sprites/sprite.svg#gin-view

I've tested in Safari and Firefox on MacOS... both look that way (see screenshot noicons.png)... checked in the gin themes folder, the sprite.svg is actually there and so is the view tag with the id of gin-view

saschaeggi’s picture

I messed up something in the webpack config. It is fixed by now :)

saschaeggi’s picture

Status: Needs work » Needs review
rkoller’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
64.04 KB

there aren't any errors loading anymore and the icons are showing again (see screenshot)

saschaeggi’s picture

Status: Reviewed & tested by the community » Fixed

Thanks!

Status: Fixed » Closed (fixed)

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