Header: Use cargo.png instead of redundant second image file with u…

…nwanted padding (#5590)
rust-lang · Dec 4, 2022 · 3cec6dc · 3cec6dc
1 parent 4075c7c
commit 3cec6dc
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 7 deletions.
diff --git a/app/components/header.hbs b/app/components/header.hbs
@@ -1,7 +1,7 @@
 <header local-class="header {{if @hero "hero"}}">
   <div local-class="header-inner">
     <LinkTo @route="index" local-class="index-link">
-      <img src="/assets/Cargo-Logo-Small.png" role="presentation" alt="" local-class="logo">
+      <img src="/cargo.png" role="presentation" alt="" local-class="logo">
       <h1>crates.io</h1>
     </LinkTo>
 

diff --git a/app/components/header.module.css b/app/components/header.module.css
@@ -12,7 +12,7 @@
         "logo search nav" auto /
         auto 1fr auto;
     align-items: center;
-    padding: var(--space-2xs) var(--space-m);
+    padding: var(--space-xs) var(--space-m);
     color: white;
 
     a {
@@ -60,11 +60,9 @@
 }
 
 .logo {
-    width: var(--space-xl);
-    height: auto;
-    /* negative margin to account for blank space in the image */
-    margin-left: -10px;
-    margin-right: calc(var(--space-s) - 10px);
+    width: auto;
+    height: calc(var(--space-m) * 1.4);
+    margin-right: var(--space-xs);
 }
 
 .search-form {

diff --git a/public/assets/Cargo-Logo-Small.png b/public/assets/Cargo-Logo-Small.png