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

Update Header.module.css #724

Closed
wants to merge 1 commit into from
Closed

Update Header.module.css #724

wants to merge 1 commit into from

Conversation

jcottam
Copy link

@jcottam jcottam commented Apr 29, 2024

Refine styling of Header to match font-size to other header elements. Cleaner look.

Refine styling of Header to match font-size to other header elements.
@jcottam
Copy link
Author

jcottam commented Apr 29, 2024

Screenshot 2024-04-29 at 12 12 34 PM

@felixmosh
Copy link
Owner

Hi @jcottam what issue did you solved?
I'm not sure what was the issue

@jcottam
Copy link
Author

jcottam commented May 1, 2024

Hi @felixmosh. The update is purely cosmetic, no bug fix or feature improvement here. I modified the CSS of the "Bull Dashboard" header to have a smaller font and left-align. I feel the update provides a cleaner look and allows for more characters in the header if needed.

Header is quite large in the current release (see below):

Screenshot 2024-04-30 at 9 27 38 PM

@felixmosh
Copy link
Owner

Hi, thanx for the PR, but looks like you are not fixing any issue but changing the design to meet your teste :]

BTW, I'm using a known typo scale, so it is not just random numbers.
image

Maybe It is possible to "jump" one layer below... to 1.44rem

@felixmosh felixmosh closed this in d920e47 May 1, 2024
@jcottam
Copy link
Author

jcottam commented May 1, 2024

1.44rem looks good! Thanks for sharing the extra context.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants