From a78d0bfcdff9dd70a804445196c041e87650054d Mon Sep 17 00:00:00 2001 From: sadick Date: Wed, 19 Jun 2019 16:32:40 +0300 Subject: [PATCH] Fix merge conflicts --- .../src/__snapshots__/index.test.jsx.snap | 57 +++++++++++-------- .../components/psammead-brand/src/index.jsx | 5 +- 2 files changed, 35 insertions(+), 27 deletions(-) diff --git a/packages/components/psammead-brand/src/__snapshots__/index.test.jsx.snap b/packages/components/psammead-brand/src/__snapshots__/index.test.jsx.snap index 2b0d67c905..07959eef37 100644 --- a/packages/components/psammead-brand/src/__snapshots__/index.test.jsx.snap +++ b/packages/components/psammead-brand/src/__snapshots__/index.test.jsx.snap @@ -301,7 +301,7 @@ exports[`Brand should render correctly with no service Localised Name 1`] = ` `; exports[`Brand should render correctly with transparent borders 1`] = ` -.c2 { +.c3 { -webkit-clip-path: inset(100%); clip-path: inset(100%); -webkit-clip: rect(1px,1px,1px,1px); @@ -312,6 +312,11 @@ exports[`Brand should render correctly with transparent borders 1`] = ` width: 1px; } +.c1 { + max-width: 80rem; + margin: 0 auto; +} + .c0 { background-color: #B80000; height: 3.5rem; @@ -321,7 +326,7 @@ exports[`Brand should render correctly with transparent borders 1`] = ` border-bottom: 0.0625rem solid transparent; } -.c1 { +.c2 { box-sizing: content-box; fill: #FFFFFF; padding-top: 1rem; @@ -331,8 +336,8 @@ exports[`Brand should render correctly with transparent borders 1`] = ` min-width: 11.25rem; } -.c3:hover .c1, -.c3:focus .c1 { +.c4:hover .c2, +.c4:focus .c2 { -webkit-text-decoration: none; text-decoration: none; border-bottom: 0.25rem solid #FFFFFF; @@ -346,14 +351,14 @@ exports[`Brand should render correctly with transparent borders 1`] = ` } @media (min-width:37.5rem) { - .c1 { + .c2 { padding-top: 1.75rem; padding-bottom: 1.5rem; } } @media screen and (-ms-high-contrast:active),print { - .c1 { + .c2 { fill: windowText; } } @@ -361,26 +366,30 @@ exports[`Brand should render correctly with transparent borders 1`] = `
- - - BBC News - + + + + + + BBC News + +
`; diff --git a/packages/components/psammead-brand/src/index.jsx b/packages/components/psammead-brand/src/index.jsx index a12962de71..3627cd2262 100644 --- a/packages/components/psammead-brand/src/index.jsx +++ b/packages/components/psammead-brand/src/index.jsx @@ -22,7 +22,6 @@ const PADDING_AROUND_SVG_BELOW_600PX = 32; const conditionallyRenderHeight = (svgHeight, padding) => svgHeight ? `height: ${(svgHeight + padding) / 16}rem` : ''; - const TRANSPARENT_BORDER = `0.0625rem solid transparent`; const SvgWrapper = styled.div` @@ -170,12 +169,12 @@ const Brand = props => { {url ? ( - + ) : ( - + )}