When the "Admin Toolbar Tools" sub-menu is enabled, the Drupal 8 logo at the beginning of the bar appears with sharp, pixelised edges in some browsers (eg. FF49.0.2 on OS X 10.11.4):

Screenshot and enlarged zoom view of the details

This is because the /admin_toolbar_tools/misc/icons/ffffff/d8-item.svg file contains the shapes as embedded raster image data, not native vector nodes. So I pixel-perfectly replaced it with the officially provided pathed version in Inkscape. Saved in optimized SVG also helped to significantly decrease the file size from 52 kB to 1 kB.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Balu Ertl created an issue. See original summary.

Balu Ertl’s picture

Balu Ertl’s picture

Status: Active » Needs review

Quequed for automatic testing.

Status: Needs review » Needs work

The last submitted patch, 2: 2707747-logo-antialiasing-2.patch, failed testing.

The last submitted patch, 2: 2707747-logo-antialiasing-2.patch, failed testing.

eme’s picture

Testing have issue (some ID have changed). So I think this is good.

Balu Ertl’s picture

Status: Needs work » Needs review

Now as the branch passed (#2707789: Fix Automated Tests), I added re-tests to my patches.

esod’s picture

Great. As soon as @eme merges in Fix Automated Tests, your patches and my subsequent patches will start to pass testing.

ping @eme

Status: Needs review » Needs work

The last submitted patch, 2: 2707747-logo-antialiasing-2.patch, failed testing.

Balu Ertl’s picture

Status: Needs work » Needs review
Balu Ertl’s picture

As now #2707789: Fix Automated Tests has been fixed on 4 May, now submitting the same patch hoping that it will get through the testing finally :)

Balu Ertl’s picture

Yay, passed! :) Could someone please confirm that the D8-logo appears anti-aliased for them with the patch #11 applied?

nevergone’s picture

Status: Needs review » Reviewed & tested by the community

#11 Tested and works well!

Balu Ertl’s picture

Any hope to get this improvement committed into master branch?

  • romainj committed 5e3e9f8 on 8.x-1.x authored by Balu Ertl
    Issue #2707747 by Balu Ertl, nevergone, eme, esod: D8 logo not rendered...
romainj’s picture

Commited in latest dev branch.

romainj’s picture

Status: Reviewed & tested by the community » Closed (fixed)