Page MenuHomeFreeBSD

Replace website's top banner with a higher-resolution version
ClosedPublic

Authored by mail_janek.ooo on Sun, Jun 30, 10:54 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jul 3, 11:46 AM
Unknown Object (File)
Mon, Jul 1, 1:02 AM
Unknown Object (File)
Sun, Jun 30, 5:59 PM
F87226555: after.png
Sun, Jun 30, 1:19 PM
F87226549: before.png
Sun, Jun 30, 1:19 PM
Subscribers

Details

Summary

The current banner is visibly low resolution. I made the following graphical changes to the banner:

  • upscaled by 2x
  • overlayed a high-resolution version of the FreeBSD logo which I exported from the Illustrator source file (website/static/logo/logo-basic.ai)
  • removed "FreeBSD" typography in Photoshop and placed a high-resolution version in the same place (from logo-basic.ai aswell)
  • sadly, I was not able to do the same for "The Power To Serve", since the .ai file does not contain it, and the font used for it is not available anywhere either. I therefore stayed with the original low-resolution version for that
  • I created a new ® sign
  • I applied a small patch of gradient to smooth out the line visible in the top left corner of the banner, where the image borders on the background of the rest of the navbar
BeforeAfter
before.png (1×2 px, 858 KB)
after.png (1×2 px, 864 KB)

I then made the following changes in the repository:

  • included the new banner, removed the old one
  • I decided to rename it from logo-red.png to banner-red.png, since it's more accurate, and easier to test (replacing an image file without renaming it was often causing preview issues for me, i.e. the old image was often still displayed intead of the new one)
  • I changed all instances of the usage to use the new asset and added explicit height/width where necessary

Diff Detail

Repository
R9 FreeBSD doc repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

mail_janek.ooo created this revision.
mail_janek.ooo retitled this revision from Replace logo-red.png with a higher-resolution banner-red.png; ensure html width and height are the same as with the original to Replace website's top banner (logo-red.png) with a higher-resolution version.Sun, Jun 30, 11:09 AM
mail_janek.ooo edited the summary of this revision. (Show Details)
mail_janek.ooo edited the test plan for this revision. (Show Details)
mail_janek.ooo added a reviewer: 0mp.
mail_janek.ooo edited the test plan for this revision. (Show Details)
mail_janek.ooo retitled this revision from Replace website's top banner (logo-red.png) with a higher-resolution version to Replace website's top banner with a higher-resolution version.Sun, Jun 30, 1:23 PM

The patch looks great! I'll commit it soon.

I'll try to get a second pair of eyes to take a look at the banner change.

website/themes/beastie/layouts/partials/site-header.html
4

Just a style nit: it seems like all the other elements are all on one line. Maybe we can keep it that way.

This revision is now accepted and ready to land.Mon, Jul 1, 8:44 AM
bcr added a subscriber: bcr.

Looks good to me.

website/themes/beastie/layouts/partials/site-header.html
4

Actually, I can fix that in the commit. Let me know if it makes sense to you.

website/themes/beastie/layouts/partials/site-header.html
4

absolutely; thank you for noticing and for the fix

The patch has landed! Thanks a lot!