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

Main title in banner should not be h1. It should just be big text #150

Open
2 tasks
yannickwurm opened this issue Sep 16, 2022 · 5 comments
Open
2 tasks
Assignees

Comments

@yannickwurm
Copy link
Member

yannickwurm commented Sep 16, 2022

Because this leads many pages to have two h1s, the first one of which is not unique

  • Main title in banner should not be h1. It should just be big text
  • Check (with grep?) or with checkbot.io that every page has an h1
@yuriigr
Copy link
Collaborator

yuriigr commented Sep 20, 2022

I understand this. It was a workaround for 'SEO audit' issue, related to missing h1 on multiple pages.
I can rollback h1 to p for the main banner, if needed. And use h1 banner only for Home and News pages, because they don't have any header text at all.

@yannickwurm
Copy link
Member Author

I undrestand.

I guess h1 can be added specifically inside those 2 pages (rather than in the banner?).

@yuriigr
Copy link
Collaborator

yuriigr commented Sep 23, 2022

Yes. But these two pages was only an example of page w/o title. A lot of pages have h2 or h3 as a main title.

@yannickwurm
Copy link
Member Author

yannickwurm commented Sep 24, 2022

OK yeah - so I guess we want to upgrade those titles to h1 - but there wants to be a single h1 per page

@yuriigr
Copy link
Collaborator

yuriigr commented Sep 24, 2022

Ok, no problem. I'm going to undo the banner changes. After that we will run the SEO and Checkbot.
Pls, provide me with titles for Home and News pages.

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

2 participants