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

Zoologist headstart bug #5809

Closed
MaggieCabrera opened this issue Apr 4, 2022 · 3 comments
Closed

Zoologist headstart bug #5809

MaggieCabrera opened this issue Apr 4, 2022 · 3 comments
Labels
[Theme] Zoologist [Type] Bug Something isn't working

Comments

@MaggieCabrera
Copy link
Contributor

It looks like Zoologist headstart is carrying a bug that was already fixed on the code of the theme. Mainly the extra div around the query pagination that was removed by #4805 but the headstart was never updated with this change.

To reproduce this I created a new site, starting with Zoologist as default theme and no other content than the default hello world post. I was never printed if I wanted the demo content at all and I see the double line caused by the unwanted div:

Screenshot 2022-04-04 at 10 53 43

This is likely to be affecting all Blockbase children, and any other markup that we are not updating on headstarts will make bugs persistent. We also can't simply re generate headstarts since some have been through some kind of manual editing to trim the number of posts or any other type of change. In this case the error is not breaking sites, just a minor visual bug, but this could lead to problems when you can't solve the bug for existing user sites if the site has been created through headstart.

@MaggieCabrera MaggieCabrera added [Theme] Zoologist [Type] Bug Something isn't working labels Apr 4, 2022
@jeffikus
Copy link
Contributor

Sounds like a small change. Let's keep this on the board. @pbking and I are going to work on the headstart for Pendant next week so I could perhaps look at this at the same time.

@pbking
Copy link
Contributor

pbking commented Aug 9, 2022

The change for this can be found here: D85662-code

I looked at all of the themes changed in #4805 and only Zoologist and Quadrat seemed to need adjusting.

I have modified only the en.json version. Do all languages need to be changed?

@mikachan
Copy link
Member

Closing this as D85662-code has been deployed 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Theme] Zoologist [Type] Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants