Skip to content

Commit

Permalink
fix modal behavior on Calypso
Browse files Browse the repository at this point in the history
  • Loading branch information
dhasilva committed Nov 13, 2024
1 parent da4dd0c commit 93825ba
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,7 @@ const UTMBuilder: React.FC< Props > = ( { modalClassName, trigger } ) => {
title={ translate( 'URL Builder' ) }
onRequestClose={ closeModal }
overlayClassName="stats-utm-builder__overlay"
bodyOpenClassName="stats-utm-builder__body-modal-open"
>
<div className={ clsx( modalClassName, 'stats-utm-builder-modal' ) }>
<div className="stats-utm-builder__fields">
Expand Down
5 changes: 5 additions & 0 deletions client/my-sites/stats/stats-module-utm-builder/style.scss
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,11 @@ $stats-utm-builder-help-padding: 38px;
$stats-utm-builder-vertical-spacing: 24px;
$modal-content-padding: 32px;

.stats-utm-builder__body-modal-open {
// Not added by default on Calypso
overflow: hidden;
}

.stats-utm-builder__overlay {
@media (min-width: $break-large) {
.components-modal__frame {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,8 +47,7 @@ function getEnvStatsFeatureSupportChecks( state: object, siteId: number | null )
),
supportsUTMStats:
// UTM stats are only available for Jetpack sites for now.
// isSiteJetpackNotAtomic,
true,
isSiteJetpackNotAtomic,
supportsDevicesStats: isSiteJetpackNotAtomic,
supportsOnDemandCommercialClassification: version_greater_than_or_equal(
statsAdminVersion,
Expand Down

0 comments on commit 93825ba

Please sign in to comment.