Guadalupe Camacho
Guadalupe Camacho
@brandonkelly I thought it might be helpful in case we needed to use it elsewhere in the code (think I may have been playing around with something like that on...
@brandonkelly just tested this and setting `aria-labelledby` on these inputs ended up causing some problems, as it's overriding the `aria-label` attribute. We should definitely move the formatting reference ID to...
Looks great, @brandonkelly!
@brandonkelly just tested and made the following changes: - "Development Settings" was added as a translation, and used as the legend for the dev-related checkbox group - Removed the inner-nested...
> Do you think we should change that media query to start at 600px, or drop the useMobileStyles() check down to 500px? @brandonkelly My rationale here was that at 500px...
@Leeroy-agency thanks for bringing this to our attention! Because these elements are inactive, there’s no hard requirement for them to meet the minimum contrast ratio of 4.5:1 against their background...
@brandonkelly I’m all for that extra accessibility feature regarding italic text. However, it’s still necessary to provide either a checkbox or toggle button for screen reader users. This has the...
@brandonkelly no, we don't need to list all of already-selected elements inside the modal; I was referring to selected elements that exist as rows inside the applicable table view (based...
For anyone else having this problem, I solved this by adding a new `createPage` action inside of the `postsList` loop in `gatsby-node.js` to generate the missing routes. ``` if (edge.node.data.categories[0].category)...
Using nvm to switch to 8.11.1 for Slate v0 development also worked for me. 👍 Thanks all.