Skip to content

Commit 2f5640b

Browse files
authored
fix: used a new classname for banner container earlier one was in ad block list (#8433)
* fix: used a new classname for banner container earlier one was in ad block list * chore: minor comment added * chore: minor comment added
1 parent 121debc commit 2f5640b

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

frontend/src/container/AppLayout/AppLayout.styles.scss

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,7 @@
1-
.app-banner-container {
1+
// Earlier we were having app-banner-container class
2+
// we change it to app-banner-wrapper as the adblocker was blocking the app-banner-container class
3+
// Keep an eye on What classnames are used in the codebase
4+
.app-banner-wrapper {
25
position: relative;
36
width: 100%;
47
}

frontend/src/container/AppLayout/index.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -613,7 +613,7 @@ function AppLayout(props: AppLayoutProps): JSX.Element {
613613
</Helmet>
614614

615615
{isLoggedIn && (
616-
<div className={cx('app-banner-container')}>
616+
<div className={cx('app-banner-wrapper')}>
617617
{SHOW_TRIAL_EXPIRY_BANNER && (
618618
<div className="trial-expiry-banner">
619619
You are in free trial period. Your free trial will end on{' '}

0 commit comments

Comments
 (0)