Merge pull request #1049 from Financial-Times/CON-3426-fix-ask-ft-pad…

…ding CON-3426: fix padding on ask to ft in header
Financial-Times · Jul 11, 2024 · 32fec96 · 32fec96
2 parents a2e80cc + 41e9cf9
commit 32fec96
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 10 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/dotcom-ui-header/package.json b/packages/dotcom-ui-header/package.json
@@ -33,7 +33,7 @@
   },
   "peerDependencies": {
     "@financial-times/logo-images": "^1.10.1",
-    "@financial-times/o-header": "13.0.1",
+    "@financial-times/o-header": "13.0.3",
     "n-ui-foundations": "^9.0.0 || ^10.0.0",
     "react": "17.x || 18.x",
     "react-dom": "17.x || 18.x"

diff --git a/packages/dotcom-ui-header/src/header.scss b/packages/dotcom-ui-header/src/header.scss
@@ -34,7 +34,9 @@
 .o-header__drawer-search-term {
   width: 100%;
 }
-
+.o-header__top-wrapper .ft-header__ask-ft-button {
+  padding: 8px;
+}
 .ft-header__ask-ft-button {
   @include oTypographySans(-1, $weight: 'semibold');
   color: oColorsByName('ft-grey');

diff --git a/packages/dotcom-ui-header/styles.scss b/packages/dotcom-ui-header/styles.scss
@@ -12,8 +12,6 @@ $system-code: 'page-kit-header' !default;
 
 @import "n-topic-search/main";
 @include nTopicSearch;
-.n-topic-search {
-  @include oGridRespondTo($from: 'M') {
-    margin: 24px 0 0;
-  }
+.o-header__search--primary .n-topic-search, .o-header__search--sticky .n-topic-search {
+  margin: 24px 0 0;
 }