globalbrain / sefirot

Global Brain Design System.
https://sefirot.globalbrains.com
MIT License
151 stars 12 forks source link

feat(action-list): support showing tooltip and implement disabled prop #516

Closed brc-dd closed 2 months ago

brc-dd commented 3 months ago

TODO:

netlify[bot] commented 3 months ago

Deploy Preview for sefirot-story ready!

Name Link
Latest commit 77ada71b084215a5b38c6d28f054482129d0ff24
Latest deploy log https://app.netlify.com/sites/sefirot-story/deploys/663346c65a5e8a00088becd1
Deploy Preview https://deploy-preview-516--sefirot-story.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

netlify[bot] commented 3 months ago

Deploy Preview for sefirot-docs ready!

Name Link
Latest commit 77ada71b084215a5b38c6d28f054482129d0ff24
Latest deploy log https://app.netlify.com/sites/sefirot-docs/deploys/663346c6c12e4b0008407642
Deploy Preview https://deploy-preview-516--sefirot-docs.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

brc-dd commented 2 months ago

Ah, it breaks SAvatarStack 👀 It should be inline-block by default too instead of undefined 👀 But no, something else appears to be breaking it again. I remember having fixed it earlier.

brc-dd commented 2 months ago

Fixed it. Check once and please revert the action list story. We can keep table one if you want 👀

kiaking commented 2 months ago

@brc-dd Looking good to me! I've adjusted the font size/weight on action list item, and reverted action list story. Let's keep table story as is 👍