Fix / header navigation list (accessibility)

jwplayer / ott-web-app

Reference implementation for JWP-powered apps

Apache License 2.0

69 stars 52 forks source link

Fix / header navigation list (accessibility) #491

Closed langemike closed 2 months ago

langemike commented 2 months ago

This change wraps header navigation items within a list.

Since I did not want to apply "reset" styles to the <ul> within Layout.module.scss, because it belongs in the Header.module.scss. So I refactored the Header component to make this possible. I have also moved the .headerButton CSS code to Header.module.scss to separate the concerns.

Original PR: https://github.com/Videodock/ott-web-app/pull/173

AntonLantukh commented 2 months ago

@langemike could you please resolve the conflicts?

langemike commented 2 months ago

could you please resolve the conflicts?

@AntonLantukh done