Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rainfall: try using index as home template #6701

Merged
merged 5 commits into from
Oct 31, 2022
Merged

Conversation

mikachan
Copy link
Member

Changes proposed in this Pull Request:

This moves the contents of the home.html template to index.html, in order to try out not including a home.html template.

I've also added an archive.html template, which uses the previous index.html content.

<!-- /wp:column -->
</div>
<!-- /wp:columns -->
</div>
Copy link
Member Author

Choose a reason for hiding this comment

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

These changes are just indenting the markup, so I could more easily see the differences between this file and home-query..

<!-- wp:query-pagination-previous /-->
<!-- wp:query-pagination-next /-->
<!-- /wp:query-pagination -->
</div>
Copy link
Member Author

Choose a reason for hiding this comment

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

These changes are just indents too.

@mikachan mikachan requested a review from a team October 27, 2022 14:41
Copy link
Member

@madhusudhand madhusudhand left a comment

Choose a reason for hiding this comment

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

These changes looks good.
While testing I have found the following minor issues.

  1. Search text in the search bar is same as background, so it is not visible

image

and is it intentional to not have the border between the posts in search and archive?

  1. Search/archive posts are not aligned with other content (in small screens)

image

@mikachan
Copy link
Member Author

Thanks @madhusudhand!! Good spots!

Search text in the search bar is same as background, so it is not visible

Pushed a fix here: 47b7760

Search/archive posts are not aligned with other content (in small screens)

Pushed a fix here: 10e49f6

and is it intentional to not have the border between the posts in search and archive?

No this isn't intentional, this border should show. It's a bottom border on each post element (from query.html). It's showing like this on my local site:

image

I wonder why it's not showing for you!

@madhusudhand
Copy link
Member

Thanks for addressing the issues. It looks good now.

I wonder why it's not showing for you!

Thanks for confirming the issue. After a rebuild of Gutenberg it started working. It might be due to cache.

@mikachan mikachan merged commit 4a670df into trunk Oct 31, 2022
@mikachan mikachan deleted the rainfall/home-template branch October 31, 2022 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants