bpetetot/conference-hall-beta

No headings, button used instead of link

Closed this issue · 1 comments

On the page: https://conference-hall.io/public/event/FdP7ePMBrbLq9kZd4Hye

  • The heading structure is unexpected. "AccessibilityCon", sitting over the hero image, should be an <h1>, and "Call for paper is opened" should be an <h2>. Instead, the page has no <h1> nor <h2> and the first headings are "Talk categories" and "Talk formats" each as <h3> (correct if the other headings are added).
  • The "Submit a Talk" control is a <button>, which is fine for changing something on the page (displaying content, toggling a box, popping a modal). It is used to navigate the user to a new page, however, so it should be a link.

For more details and guidance, see:

.

Thanks again for the feedback 😉 🙏

As you recommend, we applied a better heading structure for the page.
Fix: ee74ee9

Submit talk has been replaced by a link (instead of a button)
Fix: 3a5f6fe