Skip to content
This repository has been archived by the owner on Oct 28, 2022. It is now read-only.

Update home template #111

Merged
merged 11 commits into from
Aug 26, 2022
Merged

Update home template #111

merged 11 commits into from
Aug 26, 2022

Conversation

mikachan
Copy link
Member

This PR removes a section from the home template and tidies up the spacing, based on recent spacing preset changes:

@mikachan mikachan added the [Type] Enhancement New feature or request label Aug 24, 2022
@mikachan mikachan mentioned this pull request Aug 24, 2022
@mikachan mikachan linked an issue Aug 24, 2022 that may be closed by this pull request
@mikachan mikachan linked an issue Aug 24, 2022 that may be closed by this pull request
@carolinan
Copy link
Collaborator

Should this also include the query pagination?

@mikachan
Copy link
Member Author

Should this also include the query pagination?

Yes, thank you!

templates/home.html Outdated Show resolved Hide resolved
templates/home.html Outdated Show resolved Hide resolved
templates/home.html Outdated Show resolved Hide resolved
@carolinan
Copy link
Collaborator

The separator above the query pagination is missing.

With background color on the query, (screenshot from the editor):
missing space between the posts and the pagination

@mikachan
Copy link
Member Author

The separator above the query pagination is missing.

Thanks! I've added a spacer block here, which matches the spacing below the pagination.

@carolinan
Copy link
Collaborator

The separator is still missing.

@mikachan
Copy link
Member Author

The separator is still missing.

Ohh I see 😅 I've swapped the spacer with a separator now.

image

templates/home.html Outdated Show resolved Hide resolved
Co-authored-by: Carolina Nymark <hi@themesbycarolina.com>
@mikachan
Copy link
Member Author

🎉 Thanks for your patience with this one!

@mikachan mikachan merged commit d80753d into trunk Aug 26, 2022
@mikachan mikachan deleted the update/home-template branch August 26, 2022 15:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
[Type] Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Home.html: Typo in Heading Home template: match the Figma
3 participants