Merge pull request #1047 from Financial-Times/CON-3426-DesignChangesO…

…nTopSearchBar CON-3426: design changes ask button, edition switcher, subscribe button
Financial-Times · Jul 8, 2024 · 7b4a8da · 7b4a8da
2 parents edd2b28 + 0e2c980
commit 7b4a8da
Show file tree

Hide file tree

Showing 14 changed files with 763 additions and 626 deletions.
diff --git a/examples/ft-ui/__test__/integration.test.js b/examples/ft-ui/__test__/integration.test.js
@@ -61,7 +61,7 @@ describe('examples/ft-ui', () => {
     })
 
     it('renders the drawer menu', async () => {
-      await expect(page).toMatchElement('#o-header-drawer', { text: 'Switch to International Edition' })
+      await expect(page).toMatchElement('#o-header-drawer', { text: 'International' })
       await expect(page).toMatchElement(
         '#o-header-drawer',
         { text: 'Top sections' },

diff --git a/examples/kitchen-sink/__test__/integration.test.js b/examples/kitchen-sink/__test__/integration.test.js
@@ -47,7 +47,7 @@ describe('examples/kitchen-sink/integration', () => {
   })
 
   it('renders edition with current edition selected', () => {
-    expect(response.text).toContain('<p class="o-header__drawer-current-edition">International Edition</p>')
+    expect(response.text).toContain('<span class="o-header__drawer-menu-item o-header__drawer-current-edition">International</span>')
   })
 
   it('renders app context data as embedded JSON', () => {

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
@@ -30,10 +30,10 @@
     "@svgr/core": "^5.0.0",
     "camelcase": "^6.0.0",
     "check-engine": "^1.10.1",
-    "@financial-times/o-header": "^12.0.0"
+    "@financial-times/o-header": "^13.0.0"
   },
   "peerDependencies": {
-    "@financial-times/o-header": "^12.0.0",
+    "@financial-times/o-header": "^13.0.0",
     "@financial-times/logo-images": "^1.10.1",
     "react": "17.x || 18.x",
     "react-dom": "17.x || 18.x",