[Audit] Tooltip

[joshblack]

Interactive Tooltip

• Button should not have vertical padding

Expected:

Actual:

• Label should be label-01, #565656

• Icon should use #565656, what's the token @jeanservaas?

• What should the vertical distance be between the tooltip box and the icon? Seems like it's 13px in the spec but 16px in the implementation cc @jeanservaas

• Test needs to use body-long-01

Icon Tooltip

• Direction of tooltip seems flipped, should go down but is going up
• Height should be 24px, right now it's 32px
• Text should use body-short-01 token

Definition Tooltip

• Tooltip background should not have right padding

Expected:

Actual:

• Tooltip text should be centered

• What type style should this use @jeanservaas? body-short-01? In spec it's IBM Type C

[jeanservaas]

@joshblack

• Icon token for (#565656) should be $icon-02

Also, I reuploaded the spec, but I noticed another discrepancy in the spec and the changelog i think,

• definition tooltip and interactive tooltip should be calling $label-01 ( you can see all type tokens spec'd in the file now... everything inside the tooltip is $body-short-01 any type outside the tooltip... we're going to go with $label-01 right now, even for the definition tooltip. Sorry for the confusion!

• Also, is this the distance you were asking about?

[carbon-bot]

🎉 This issue has been resolved in version 9.83.19 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀