Merge pull request #2200 from CityOfPhiladelphia/new-homepage

Correct image size
CityOfPhiladelphia · Oct 24, 2024 · b618270 · b618270
2 parents d79d26e + bb717c6
commit b618270
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 12 deletions.
diff --git a/...-content/plugins/phila.gov-customization/public/controllers/class-phila-featured-news.php b/...-content/plugins/phila.gov-customization/public/controllers/class-phila-featured-news.php
@@ -117,7 +117,7 @@ public function prepare_item_for_response( $post, $request ) {
     if (isset( $schema['properties']['featured_image'] )) {
       $featured_image_id = get_post_thumbnail_id($post);
       if (wp_get_attachment_image_src($featured_image_id)) {
-        $medium_featured_image_url = wp_get_attachment_image_src($featured_image_id);
+        $medium_featured_image_url = wp_get_attachment_image_src($featured_image_id, 'medium');
         $post_data['featured_image']  = (string) $medium_featured_image_url[0];
       }
     }

diff --git a/wp/wp-content/themes/phila.gov-theme/css/scss/base/_global-nav.scss b/wp/wp-content/themes/phila.gov-theme/css/scss/base/_global-nav.scss
@@ -7,6 +7,9 @@
     width: 100%;
   }
 }
+.menu-wrapper {
+  width:100%;
+}
 
 .global-nav {
   @include secondary-font(400);
@@ -16,7 +19,7 @@
   ul {
 
     font-size: 1rem;
-    
+
     li.gov-site.show-for-medium {
       font-size: .85rem;
 
@@ -514,8 +517,8 @@
 
   .menu-icon {
     position: relative;
-    width: 70px;
-    height: 70px;
+    width: 80px;
+    height: 80px;
     padding: 1.3rem;
     margin-right: 1rem;
     margin-left: 0;
@@ -652,7 +655,7 @@
     .primary-menu .menu li > a {
         font-size: 10.5px !important; //force main menu to shrink at smaller screen size. Prevents wrapping for mayor link
         text-transform: none !important;
-        padding: 2rem 1.3rem 2rem 1rem !important;
+        padding: 2rem .5rem !important;
     }
     .logo {
       margin:0;
@@ -662,9 +665,9 @@
     }
   }
 
-  @media screen and (min-width: 750px) and (max-width: 816px) {
+  @media screen and (min-width: 750px) and (max-width: 880px) {
     a.icymi {
-      width:110px;
+      width:100px;
     }
   }
 
@@ -936,7 +939,7 @@
     button.site-search {
       color:white;
       font-size: 1rem;
-      margin-right: 1rem;
+      padding: 1rem;
 
       @include breakpoint(small only) {
         min-width: 60px;

diff --git a/wp/wp-content/themes/phila.gov-theme/css/scss/pages/_post.scss b/wp/wp-content/themes/phila.gov-theme/css/scss/pages/_post.scss
@@ -220,7 +220,7 @@
   blockquote {
     border-left: 2px solid black;
     padding-left: 15px;
-  } 
+  }
   blockquote:before {
       content: ' ' !important;
   }

diff --git a/wp/wp-content/themes/phila.gov-theme/header.php b/wp/wp-content/themes/phila.gov-theme/header.php
@@ -153,10 +153,10 @@
     </div>
     <!-- sticky/desktop nav -->
     <div id="global-sticky-nav" class="row expanded">
-      <div>
+      <div class="small-24 columns phn">
         <div class="primary-menu" data-sticky-container data-swiftype-index="false">
           <div class="grid-container phila-sticky phn" data-margin-top="0" data-sticky data-sticky-on="medium">
-            <div class="grid-x align-middle align-justify">
+            <div class="menu-wrapper grid-x align-middle align-justify">
               <div class="small-20 medium-12 branding-container cell">
                 <button aria-label="Global navigation menu" class="menu-icon" type="button">
                     <i class="fass fa-bars" aria-hidden="true"></i>
@@ -194,7 +194,7 @@
           </div><!-- close columns -->
         </div>
       </div>
-    </div>
+                </div>
     <?php include(locate_template('partials/global/translations-modal.php')); ?>
     <div id="phila-mobile-menu"></div>
     <div id="phila-site-wide-alerts"></div>