Layout changes accessibility review #115
Conversation
to accomodate for two new fields
leaving rest for another branch/PR tbc
…or into layout-change
…r into layout-change
This reverts commit 2c6a9dd.
removed volume component and added titles tags to player controls btns
mobile break point
eg linting, css etc.. from James' PR feedback
+ Range bar re-styled
|
Hey @ivanji, thanks so much for this! Digging the css grid work. Was just reviewing this and adding a few things, can keep you in the loop on this branch. |
|
Appreciate the ProgressBar styling revamp - but we've been meaning to actually have it more akin to the UX mockups produced previously (attached below).
We're trying to keep some visual consistency with other BBC apps out there so will go with this style for now - I've updated the CSS accordingly. |
This looks nice! wish I had seen this mock-up before. |
Describe what the PR does
Just continuing on from #111 and cleaning things up, with a few fixes.
State whether the PR is ready for review or whether it needs extra work
Currently a work in progress