Skip to content

Commit

Permalink
Fix JP logo misalignment in the DataViews Properties list.
Browse files Browse the repository at this point in the history
  • Loading branch information
allilevine committed Sep 20, 2024
1 parent 9b970c6 commit 9e32486
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 9 deletions.
7 changes: 0 additions & 7 deletions client/components/dataviews/style.scss
Original file line number Diff line number Diff line change
Expand Up @@ -116,13 +116,6 @@
text-transform: uppercase;
}

svg {
&:not(:last-child) {
margin-right: 4px;
vertical-align: middle;
}
}

.dataviews-view-table-header-button {
color: inherit;
}
Expand Down
4 changes: 2 additions & 2 deletions client/hosting/sites/components/sites-dataviews/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -161,10 +161,10 @@ const DotcomSitesDataViews = ( {
id: 'stats',
// @ts-expect-error -- Need to fix the label type upstream in @wordpress/dataviews to support React elements.
label: (
<>
<span className="sites-dataviews__stats-label">
<JetpackLogo size={ 16 } />
<span>{ __( 'Stats' ) }</span>
</>
</span>
),
render: ( { item }: { item: SiteExcerptData } ) => <SiteStats site={ item } />,
enableHiding: false,
Expand Down
7 changes: 7 additions & 0 deletions client/hosting/sites/components/sites-dataviews/style.scss
Original file line number Diff line number Diff line change
Expand Up @@ -134,3 +134,10 @@
.sites-overview__content .dataviews-wrapper .dataviews-filters__view-actions .components-h-stack button:nth-of-type(3) {
display: none;
}

// Style the Stats label that includes the Jetpack logo
.sites-dataviews__stats-label {
display: flex;
align-items: center;
gap: 4px;
}

0 comments on commit 9e32486

Please sign in to comment.