ClickHouse / ClickHouse/click-ui

Define disabled elements via opacity rather than color

Open
#618 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
135
Forks
33
Avg merge
2d 13h
Merged PRs (30d)
19

Description

Having a “disabled” text color is not the best approach. A disabled color varies depending on where an item is displayed. And as a rule, “disabled” is not a color, disabled items have low contrast. So it's easier and more convenient to use opacity rather than color.

## Suggested change
- review disabled states of click-ui components
- for text, stop using disabled color, and use opacity instead
- for backgrounds and other visual attributes, review each case. Use opacity where possible, otherwise keep the color
- review the changes in both light and dark themes

_Originally posted by @ariser in https://github.com/ClickHouse/click-ui/issues/610#issuecomment-2980903330_

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.