Dark mode: Spinners

[julien-deramond]

New Sass vars:

• $spinner-color

⚠️ Added temporary data-bs-theme="dark" for buttons containing spinners. This should go if we modify color: automatically with [data-bs-theme].

Live previews

• https://deploy-preview-2290--boosted.netlify.app/docs/5.3/components/spinners/
• https://deploy-preview-2290--boosted.netlify.app/docs/5.3/dark-mode/#spinners

[netlify]

✅ Deploy Preview for boosted ready!

Name	Link
🔨 Latest commit	09cec75
🔍 Latest deploy log	https://app.netlify.com/sites/boosted/deploys/6524f42efa64a30008903cbb
😎 Deploy Preview	https://deploy-preview-2290--boosted.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[louismaximepiton]

Just wondering if we should recheck this once we merge text utilities since the orange used here is #f16e00 everywhere.
There's an issue here, do you think we should care about this right know ?

[julien-deramond]

I can add a sub-task to re-check that for text utilities and buttons yes. Right now, there's nothing else I can do without breaking other stuff.

Edit: I'll add a sub-task to re-check that for text utilities and a sub-task to remove data-bs-theme="dark" from spinners used alongside buttons if/when we decide to force the color: for [data-bs-theme] (it would be automatically modified).

[louismaximepiton]

I'm fine with the changes, the actual solution for spinners inside buttons might only work for light theme documentation once buttons will be handled but let's try it this way and document to recheck once finished.