Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bug(Field.Expiry): has wrong font color during disabled state #3022

Closed
joakbjerk opened this issue Dec 7, 2023 · 1 comment · Fixed by #3215
Closed

bug(Field.Expiry): has wrong font color during disabled state #3022

joakbjerk opened this issue Dec 7, 2023 · 1 comment · Fixed by #3215
Assignees
Milestone

Comments

@joakbjerk
Copy link
Contributor

The font color should be grayish like the border, instead of the regular text colors as it has now

Screenshot 2023-12-07 at 14 29 36
@langz
Copy link
Contributor

langz commented Dec 8, 2023

I created a CSB, using latest on main as of today, where I've added all our field components WITHOUT a value in disabled state https://codesandbox.io/p/sandbox/eufemia-starter-forked-gn3cg7?file=%2Fsrc%2FApp.tsx%3A58%2C23

Here's also a CSB, using latest on main as of today, where I've added all our field components WITH a value in disabled state https://codesandbox.io/p/sandbox/eufemia-starter-forked-6wry6m?file=%2Fsrc%2FApp.tsx

The purpose of making these CSBs is to be able to see how the Field components behave in these two different states, disabled with value, and disabled without value. Please report any findings and/or create issues based on this 🙏

A quick observation is that Field.Date looks different than the other fields in these states.
And that Field.Selection behaves strange 😢

I've created the following bugs/issues for these observations:

@langz langz closed this as completed Dec 8, 2023
@langz langz reopened this Dec 8, 2023
@joakbjerk joakbjerk added this to the Forms Issue milestone Jan 11, 2024
@joakbjerk joakbjerk self-assigned this Jan 11, 2024
@joakbjerk joakbjerk linked a pull request Jan 11, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

2 participants