insightsengineering / teal

Exploratory Web Apps for Analyzing Clinical Trial Data
https://insightsengineering.github.io/teal/
Other
176 stars 35 forks source link

[Question]: button styles #1242

Open chlebowa opened 3 months ago

chlebowa commented 3 months ago

What is your question?

There are plenty of buttons in a teal app. Some of them react to theme changes, while others maintain their style regardless of theme.

These buttons can be themed:
These buttons cannot be themed:

All of this causes something of a style inconsistency.

QUESTION

Is this deliberate? If not, are there plans to unify this? If so, in which direction?

Code of Conduct

Contribution Guidelines

Security Policy

donyunardi commented 3 months ago

I think the button should react to theme changes. I don't believe this is intentional, so let's plan to address it.

First, we need to identify which elements are not reacting to theme changes and find out why (e.g., is it the way we coded it?). Then, we can implement the necessary changes.

What do you think?

donyunardi commented 3 months ago

Also, maybe this is a duplicate of https://github.com/insightsengineering/teal/issues/1233?

chlebowa commented 3 months ago

First, we need to identify which elements are not reacting to theme changes and find out why (e.g., is it the way we coded it?). Then, we can implement the necessary changes.

What do you think?

Sounds good to me 👍 I identified two groups but there might be more buried in some modules, I guess.

Also, maybe this is a duplicate of #1233?

I don't believe so.