question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

[Carbon X] $inverse01 token for Button's font & icon color is bad for g90 & g100 themes

See original GitHub issue

Detailed description

There are 2 dark theme UIs provided by @carbon/themes called g90 & g100https://carbon-elements.netlify.com/themes/examples/preview/

The $inverse01 token for both of those themes is the gray100 color.

The button--theme--x mixin for both primary and secondary Carbon X buttons applied $inverse01 as the font (and icon) color. (see: https://github.com/IBM/carbon-components/blob/master/src/components/button/_button.scss#L203)

That means that buttons sitting on a dark UI background (like a gray90 or gray100) background also have gray100 font & icon color, like this:

screen shot 2019-01-29 at 9 13 04 am

We are using a dark UI in our application so I’m happy to help you test alternative token applications here.

Issue Analytics

  • State:closed
  • Created 5 years ago
  • Comments:5 (4 by maintainers)

github_iconTop GitHub Comments

2reactions
jendownscommented, Feb 1, 2019

Thanks @aagonzales! So unfortunately I don’t have access to that boxnote even with my IBMid, but I did go ahead and update text + icon color for primary and secondary Carbon X buttons in this PR, based on your comment: https://github.com/IBM/carbon-components/pull/1720

2reactions
aagonzalescommented, Jan 31, 2019

Yeah this is another token bug. It should be one of the new tokens we recently add, $text-04 for both the icon and text.

I’ve got a token change log here that should have all the correct tokens on them: https://ibm.box.com/s/3prwmp421wu6umgayvejehiy0r1hi9m4

Read more comments on GitHub >

github_iconTop Results From Across the Web

[Carbon X] $inverse01 token for Button's font & icon color is bad for ...
The $inverse01 token for both of those themes is the gray100 color. The button--theme--x mixin for both primary and secondary Carbon X buttons...
Read more >
Color - Carbon Design System
Token Role Value $ui‑background Default page background White; —; #ffffff $interactive‑01 Primary interactive color;; Primary buttons Blue 60; —; #0f62fe $interactive‑02 Secondary interactive color;; Secondary button...
Read more >
Color - Carbon Design System
Token Role Value $ui‑background Default page background White; —; #ffffff $interactive‑01 Primary interactive color;; Primary buttons Blue 60; —; #0f62fe $interactive‑02 Secondary interactive color;; Secondary button...
Read more >
How to style icon color, size, and shadow of FontAwesome Icons
Given that they're simply fonts, then you should be able to style them as fonts: #elementID { color: #fff; text-shadow: 1px 1px 1px...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found