2024-06-27 UTC
# [Joe_Crawford] It's looking pretty great. Your decision to hide (with no recourse) items inside the .hide-on-mobile class is not great for accessibility. Might consider whether to force those to display on a user action. A quibble with the markup is that technically inside an `<ul>` element only list items are allowed as direct children, (you have p and div elements) but it is HIGHLY readable and very clean. Much to like out of the gate. Nice work!