Skip to content

Fix leading zero having different font #324

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

Merged
merged 1 commit into from
Apr 30, 2024

Conversation

AaronWatson2975
Copy link
Contributor

Description

I'd like to start out by saying thanks for putting together this awesome library!

I ran into an intriguing glitch using this library in a project where we have custom fonts. There was inconsistency in the font display for the leading zero in the time input. With some research, I discovered that 'font' for the leading zero was not inherited from its parent.

To solve this, I've added the CSS rule font: inherit to the __inputGroup__leadingZero class. This aligns the font display of the leading zero with the rest of the input. I believe this fix will make the component more consistent when it comes to font styling.

Images

Before

Screenshot 2023-09-04 at 1 13 32 AM

After

Screenshot 2023-09-04 at 1 13 12 AM

This ensures the styling consistency between all elements of the component, providing a more predictable experience for future users.

@wojtekmaj wojtekmaj force-pushed the main branch 2 times, most recently from a067b23 to 5e58d95 Compare October 23, 2023 10:21
@wojtekmaj wojtekmaj force-pushed the main branch 2 times, most recently from acd6c81 to a337cde Compare November 8, 2023 11:45
@wojtekmaj wojtekmaj changed the title Fixed issue with leading zero having different font Fix leading zero having different font Apr 30, 2024
@wojtekmaj
Copy link
Owner

Thanks!

@wojtekmaj wojtekmaj merged commit a3b1fac into wojtekmaj:main Apr 30, 2024
@wojtekmaj wojtekmaj added the bug Something isn't working label Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants