-
Notifications
You must be signed in to change notification settings - Fork 69
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
Different input fields have slightly different border colors. #21
Comments
Color is the same as well as border width. The difference is that spinner uses border color (stroke) while text inputs use background insets (two rectangles one on top of another). The latter is preferred option because of performance. Nor sure why I implemented spinner CSS this way, it needs some investigation. |
Maybe that is the problem? The border color is semitransparent, isn't it?. Background insets approach is mixing the border color with container's background (unless you have an another rectangle under the border), while border color approach is mixing it with the control's background? The problem, then, of course, the container and the control have both same background color :) |
I delved into the code a bit today. The issue with the editable combo-box was that both the combo-box itself and its text field had borders, so the border was being drawn twice. Setting text-field border transparent fixed it. |
Thanks for investigating this! I think I'll also remove border opacity. At least from the Nord. |
Examples use Nord Dark color theme.
More noticeable on an editable combo-box:
Using a color picker, arrow button border reads #5d6575, while text input border reads #6e7787.
The text was updated successfully, but these errors were encountered: