Christian Oliff

Results 245 comments of Christian Oliff

thanks! I have a BrowserStack subscription and just tested this with Chrome 60 on Windows 10 and it does indeed still have an unwanted blue outline without the `outline: 0`...

All looks good to me- happy to see the browser support updated! One minor thing, `not Explorer

> @coliff and @prashanthr6383 Have we added the skip to content link to the Nav Bar? No, in order to add it we'd need to know what the skip-to-content link...

ESLint is already included in H5BP in the project root... however, it's the older v8 release. We would appreciate a PR updating to ESlint v9. For this change the existing...

This isn't super straightforward as I had originally anticipated. Our tables aren't formatted so we need another plugin. Details here: https://storybook.js.org/docs/writing-docs/mdx#markdown-tables-arent-rendering-correctly

It fails with this change: ![image](https://github.com/user-attachments/assets/6b2de5de-ed67-471c-a27e-26e43ef5008d) ![image](https://github.com/user-attachments/assets/54a8df5e-8157-4b15-af29-798a33ab6692)

A simple fix if I were in your situation would be to add a `.prettierignore` file in the root with: ``` *.yaml ```

I think the class name should be `text-balance` rather than `text-balanced` Then it exactly matches the text-wrap property value. FWIW; Tailwind use `text-balance` too: https://tailwindcss.com/docs/text-wrap#quick-reference (I was considering suggesting `text-wrap-balance`,...

> text balance is the same with text-nowrap? no, it's different. Check https://developer.chrome.com/docs/css-ui/css-text-wrap-balance

sorry, don't think I'll be able to do this anytime soon. I'd be happy for someone else to jump it and take this on though!