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

Consistent fonts on Dashboard screen #6547

Open
ekidneyrh opened this issue Mar 27, 2024 · 2 comments · Fixed by #6706 or #6685 · May be fixed by #6555
Open

Consistent fonts on Dashboard screen #6547

ekidneyrh opened this issue Mar 27, 2024 · 2 comments · Fixed by #6706 or #6685 · May be fixed by #6555
Assignees
Labels
area/ui kind/enhancement ✨ Issue for requesting an improvement

Comments

@ekidneyrh
Copy link
Contributor

ekidneyrh commented Mar 27, 2024

Is your enhancement related to a problem? Please describe

At the moment, the text on the dashboard screen is not following a consistent set of rules. The visual hierarchy needs to be stronger.

This relates to #6405

Describe the solution you'd like

Go through the page and set the font to the appropriate size, setting them up to be edited later.

Describe alternatives you've considered

No response

Additional context

From this we can go through the other pages and make them follow this format also.

@cdrage
Copy link
Contributor

cdrage commented Apr 10, 2024

I opened up issue #6747 because I couldn't find this one. Looks like it was closed by mistake.

Opening this as #6685 unfortunately did not solve the issue of font sizing throughout the dashboard screen.

@ekidneyrh
Copy link
Contributor Author

Hi @cdrage! I read the description on the other ticket. I was trying to assign the font size eg. xs, sm, base etc. for diff components so when I edit the config file last it wont make things go funky. I was also trying to play with the font weights to make it look better in the mean time but I agree with what you're saying and I'll go back in to make changes.

For the provider card, there's this ticket that was made to change how that looks, which imo makes it cleaner / simplistic and there's less types of information on the card.

Here's a screenshot of the proposed provider card from that ticket:

image

I know time is tight so I'll try tidy up what is there atm.

Thanks for your feedback!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment