Ensure Header component uses proper semantic `header` HTML tag

[Caixetadev]

I have a question, why is the Header component not a header tag? I also noticed that it has more elements that could be other elements instead of a div.

[lesliecdubs]

Hello @Caixetadev 👋🏻 Which component/file was this code in?

[Caixetadev]

Hello @Caixetadev 👋🏻 Which component/file was this code in?

here:
src/Header.tsx

[Caixetadev]

Can I open a PR for this? @lesliecdubs

[lesliecdubs]

Hi @Caixetadev. As it turns out, this component isn't in use at GitHub and is likely to be deprecated in the near future. That being said, if you'd like to submit a PR for the change, feel free.

[amarmanhala]

👋 Hi @lesliecdubs, Can I fix this issue and contribute to the project?

[lesliecdubs]

Hi @amarmanhala 👋🏻 Yes, this is still a valid issue. As per #2870 (comment) this component is likely to be deprecated in the future, but if you'd like to submit a fix in the meantime we will review the contribution.

For more details on contributing, we invite you to review the contributor docs. Thanks for participating!

[amarmanhala]

Thank you @lesliecdubs for the opportunity. I have started working on this issue.

[github-actions]

Uh oh! @Caixetadev, the image you shared is missing helpful alt text. Check your issue body.

Alt text is an invisible description that helps screen readers describe images to blind or low-vision users. If you are using markdown to display images, add your alt text inside the brackets of the markdown image.

Learn more about alt text at Basic writing and formatting syntax: images on GitHub Docs.