@@ -135,14 +135,6 @@ const DesignableBanner: ReactComponent<BannerRenderProps> = ({
135135} : BannerRenderProps ) : JSX . Element => {
136136 const isTabletOrAbove = useMatchMedia ( removeMediaRulePrefix ( from . tablet ) ) ;
137137
138- // We can use this to shorten the banner if the "open in app" banner is present
139- const [ iosAppBannerPresent , setIosAppBannerPresent ] = useState ( false ) ;
140- useEffect ( ( ) => {
141- setIosAppBannerPresent (
142- window . innerHeight != window . document . documentElement . clientHeight ,
143- ) ;
144- } , [ ] ) ;
145-
146138 const bannerRef = useRef < HTMLDivElement > ( null ) ;
147139
148140 useEffect ( ( ) => {
@@ -151,21 +143,6 @@ const DesignableBanner: ReactComponent<BannerRenderProps> = ({
151143 }
152144 } , [ ] ) ;
153145
154- useEffect ( ( ) => {
155- if ( iosAppBannerPresent ) {
156- // send ophan event
157- if ( submitComponentEvent ) {
158- submitComponentEvent ( {
159- component : {
160- componentType : 'ACQUISITIONS_OTHER' ,
161- id : 'safari-ios-banner-present' ,
162- } ,
163- action : 'VIEW' ,
164- } ) ;
165- }
166- }
167- } , [ iosAppBannerPresent , submitComponentEvent ] ) ;
168-
169146 const choiceCards = getChoiceCards ( isTabletOrAbove , choiceCardsSettings ) ;
170147 const defaultChoiceCard = choiceCards ?. find ( ( cc ) => cc . isDefault ) ;
171148
@@ -321,7 +298,6 @@ const DesignableBanner: ReactComponent<BannerRenderProps> = ({
321298 tabIndex = { - 1 }
322299 css = { styles . outerContainer (
323300 templateSettings . containerSettings . backgroundColour ,
324- iosAppBannerPresent ,
325301 templateSettings . containerSettings . textColor ,
326302 ) }
327303 className = { contextClassName }
@@ -555,15 +531,14 @@ const DesignableBanner: ReactComponent<BannerRenderProps> = ({
555531const phabletContentMaxWidth = '492px' ;
556532
557533const styles = {
558- outerContainer : (
559- background : string ,
560- limitHeight : boolean ,
561- textColor : string = 'inherit' ,
562- ) => css `
534+ outerContainer : ( background : string , textColor : string = 'inherit' ) => css `
563535 background : ${ background } ;
564536 color : ${ textColor } ;
565537 bottom : 0px ;
566- ${ limitHeight ? 'max-height: 60vh;' : '' }
538+ /* override vh with vsh if supported */
539+ max-height : 65vh ;
540+ /* stylelint-disable-next-line declaration-block-no-duplicate-properties */
541+ max-height : 65svh ;
567542
568543 * {
569544 box-sizing : border-box;
0 commit comments