[Discover] Field type icon without contrast

[l-suarez]

Accent color on light background doesn't have enough contrast:

[elasticmachine]

Pinging @elastic/kibana-data-discovery (Team:DataDiscovery)

[kertal]

@l-suarez I assume this has to be fixed on EUI side? of should we change the background of the field list? FYI @jughosta

[jughosta]

I think the icon we get is already colored by Eui https://eui.elastic.co/#/display/icons#tokens

[kertal]

@jughosta thx!
so we've got 2 options

1. EUI changes the icon
2. We change the background

@l-suarez wdyt?

[l-suarez]

Thanks team, it looks like it might be the EUI team, I'm in talks in slack atm :)

[l-suarez]

Latest news: these colors are not preset/provided by EuiToken (it accepts any color with the docs suggesting data viz colors), so this would be a Kibana (implementor) level change

[jughosta]

For consistency it would make more sense if Eui would manage (continue managing) the mapping between the token and its color. It's not only Discover where it's being used without fill property.

Is not it implemented in https://github.com/elastic/eui/blob/main/packages/eui/src/components/token/token_map.ts ?

[davismcphee]

++ to what @jughosta said. These tokens are currently used across Kibana and ideally we'd be able to rely on the default styling to provide a good experience everywhere (and only carve out exceptions with custom styling when really needed). If it must be done Kibana side for Borealis, it should be done in a global way so all Kibana usages inherit it by default, but I still feel it would be better to be able to rely on the defaults as we do with Amsterdam.

[davismcphee]

I confirmed with @ryankeairns that this will be addressed on the EUI side, and linked their private issue to this one for reference.

[elasticmachine]

Pinging @elastic/eui-team (EUI)

[ryankeairns]

Re-opening; will close when EUI PR merges

[ryankeairns]

EUI PR merged. This will be part of the pending upgrade: #204482