Skip to content

Commit

Permalink
Update spacing + dimensions in the Site Hub (#47120)
Browse files Browse the repository at this point in the history 
* Remove some internal site hub spacing

* Icon dimensions
  • Loading branch information
@jameskoster
jameskoster authored Jan 13, 2023
1 parent 9511a6d commit ac42d04
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 4 deletions.
1 change: 1 addition & 0 deletions packages/edit-site/src/components/site-hub/index.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -100,6 +100,7 @@ const SiteHub = forwardRef(
<HStack
justify="flex-start"
className="edit-site-site-hub__text-content"
spacing="0"
>
<motion.div
className="edit-site-site-hub__view-mode-toggle-container"
Expand Down
8 changes: 7 additions & 1 deletion packages/edit-site/src/components/site-hub/style.scss
View file
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,16 @@

.edit-site-site-hub__view-mode-toggle-container {
height: $header-height;
width: $header-height;
width: $header-height + 4px;
flex-shrink: 0;
}

.edit-site-layout.is-edit-mode {
.edit-site-site-hub__view-mode-toggle-container {
width: $header-height;
}
}

.edit-site-site-hub__text-content {
// Necessary for the ellipsis to work.
overflow: hidden;
Expand Down
2 changes: 1 addition & 1 deletion packages/edit-site/src/components/site-icon/index.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ function SiteIcon( { className } ) {
) : (
<Icon
className="edit-site-site-icon__icon"
size="36px"
size="32px"
icon={ wordpress }
/>
);
Expand Down
4 changes: 2 additions & 2 deletions packages/edit-site/src/components/site-icon/style.scss
View file
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@
}

.edit-site-site-icon__image {
width: $button-size;
height: $button-size;
width: $grid-unit-40;
height: $grid-unit-40;
border-radius: $radius-block-ui;
object-fit: cover;
}

1 comment on commit ac42d04

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Flaky tests detected in ac42d04.
Some tests passed with failed attempts. The failures may not be related to this commit but are still reported for visibility. See the documentation for more information.

🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/3913557594
📝 Reported issues:

Please sign in to comment.