select input options are a little more safe for screen readers - we render a more appropriate separator option
checkbox inputs in the filter options (in the settings) get aria-labelledby their respective descriptive text
converts the itemsDiv to a ul - which is the appropriate parents for the role='listitem' shopping list items
Moved the needed updates to items to another ticket - something with the grid it's all in is stopping me from effectively adding some hidden (screen reader only) text to describe the intent of the checkbox - but the list items need to be more thoughtfully designed now that we have additional data we're rendering (store section, item quantity)
Adds
aria
labels where appropriate:select
inputoptions
are a little more safe for screen readers - we render a more appropriate separator optioncheckbox
inputs in the filter options (in the settings) getaria-labelledby
their respective descriptive textitemsDiv
to aul
- which is the appropriate parents for therole='listitem'
shopping list itemsgrid
it's all in is stopping me from effectively adding some hidden (screen reader only) text to describe the intent of the checkbox - but the list items need to be more thoughtfully designed now that we have additional data we're rendering (store section, item quantity)