Esri / calcite-design-system

A monorepo containing the packages for Esri's Calcite Design System
https://developers.arcgis.com/calcite-design-system/
Other
285 stars 76 forks source link

Scrollbar design tokens #8621

Open jmanke opened 8 months ago

jmanke commented 8 months ago

Check existing issues

Description

Currently, there is no way to customize the scrollbar style of components. In Dashboards, we have a custom scrollbar style that is globally applied. As we integrated calcite components, we observe differences between the default scrollbar styles of calcite-components, and our scrollbar styles.

We would like to control scrollbar styles as some OS + Browser's don't have visually appealing scrollbars, or they conflict with the theme the Dashboard is going for. For example, a dark themed Dashboard could have a very light colored scrollbar in Windows browsers.

cc @ffaubry @jcfranco @vijayendranj

Acceptance Criteria

A way to customize components' scrollbar styles (when they are in the shadow root) - the first thought is to use design tokens (CSS variables).

Relevant Info

No response

Which Component

Any component that has a scrollbar inside of the shadow root.

Example Use Case

Dashboards look beautiful, cohesive, and focused. Scroll bars play into that. When they don't look right (too big, not right color, different across elements), they detract from those qualities. Being able to apply scrollbars styles to all relevant calcite-components will ensure the application's style is consistent and aligns with our styling goals.

Priority impact

p4 - not time sensitive

Calcite package

Esri team

ArcGIS Dashboards

jcfranco commented 7 months ago

Discussed this with the team and we would need to wait for scrollbar-width and scrollbar-color to be widely supported across browsers for this. Safari is the only one lacking support, but it looks that might change soon based on the following WebKit PRs: https://github.com/WebKit/WebKit/pull/14597, https://github.com/WebKit/WebKit/pull/13992.

Testing on browsers that support this show these styles piercing through shadow DOM: https://codepen.io/jcfranco/pen/OJqOvgG.

macandcheese commented 4 months ago

@jcfranco maybe we could expose some non-public css variables here for Esri teams that want to adopt despite the limited browser support?

This could help reduce future breaking change cc @vijayendranj

vijayendranj commented 4 months ago

//cc @ffaubry This is something web components team may also be interested.

geospatialem commented 1 month ago

For design to consider implementation on consistency with scrollbar styling.

Of note Safari would not have support as mentioned in Franco's comment above.