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

Update Hey theme to ensure there's a way to return home before uploading a site logo #8024

Open
annezazu opened this issue Aug 20, 2024 · 4 comments

Comments

@annezazu
Copy link

When @ellatrix was going through setting up a blog on WordPress.com, she noticed that after publishing a post, there wasn't a clear way to get to the homepage due to the theme's design and the onboarding:

I’m guessing the design of the Hey theme is to have a site logo act as the home button. But there was no setup step for the site logo, so the home button was missing.

I'm opening this for consideration to either:

  1. update the Hey theme to include a site title to ensure folks can return home OR a default site logo
  2. pick a different default theme to give folks when going through this flow.
  3. change onboarding to include setting a site logo.

All options considered, updating the theme to ensure there's a site title OR even a default site logo folks can replace seems best.

@alaczek
Copy link
Contributor

alaczek commented Aug 21, 2024

Thanks for reporting @annezazu!

pick a different default theme to give folks when going through this flow.

We will be switching the default theme in the write flow to Poema to address this. (see https://github.com/Automattic/dotcom-forge/issues/8774 for details)

update the Hey theme to include a site title to ensure folks can return home OR a default site logo

I think this would be a positive change - I was actually thinking about switching the "Hey, World." text to site title block, as it seems like the most natural place for it. I am a bit concerned though, because it would be a fairly bing change to live customer sites (and it's a lot of sites, since it has been a default for a while). But then again customers would still be able to revert back to the old version by editing the template in the editor. What do you think?

We could also delist Hey, since it's a bit dated, has issues, and we're replacing it with Poema. Wouldn't fix existing sites, but it wouldn't be available for new sites anymore.

@annezazu
Copy link
Author

We could also delist Hey, since it's a bit dated, has issues, and we're replacing it with Poema. Wouldn't fix existing sites, but it wouldn't be available for new sites anymore.

I think this is a better approach then messing with prior sites and making them have to make changes! Thanks for following up here. I'll close this out since it sounds like the root problem with the theme will resolve itself when a new theme is used!

@mtias
Copy link
Member

mtias commented Sep 24, 2024

I love the design of Hey, let's not delist it.

@dsas
Copy link
Contributor

dsas commented Nov 29, 2024

There's another instance of Hey's lack of navigation back to the home page being confusing: paYJgx-5CF-p2 If we really have to keep Hey around, then I wonder if it's worth reopening this issue to improve it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants