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

Standardize font and font-size across all pages and components #49

Open
juliaferraioli opened this issue Apr 15, 2022 · 16 comments
Open
Assignees
Labels
good first issue Good for newcomers UX

Comments

@juliaferraioli
Copy link
Member

This is totally my fault, and I think we need to revert it or figure out what our default size should be. Right now it looks a little disjointed with the text on the pages and text on the components.

@juliaferraioli juliaferraioli added UX good first issue Good for newcomers labels Apr 15, 2022
@YC-Kwan
Copy link

YC-Kwan commented Apr 19, 2022

Hi there, may I know what should do about this issue?

@juliaferraioli
Copy link
Member Author

Hi @YC-Kwan, thanks for asking!

If you look at the bio text on our about page, the font and font color isn't set to what we use in the text above it.

For the data rendering, we probably want the div to have almost-black added to the class:

<div class="team-person">

the h2 to have f3 fw1 lh-title added to the class:

<h2>{{ .name }}, {{ .roles }}</h2>

and the p to have f5-l nested-copy-line-height added to the class:

There may be other areas on the site that you notice similar inconsistencies -- please let me know if this is an issue you'd like to take on!

@YC-Kwan
Copy link

YC-Kwan commented Apr 21, 2022

Okay, thanks for the information but for the data rendering what does it mean to have almost-black in the class?

@juliaferraioli
Copy link
Member Author

@YC-Kwan great question! almost-black is in our css:

It turns the font color nearly black, or #222222 if you prefer the hex.

@YC-Kwan
Copy link

YC-Kwan commented Apr 24, 2022

Okay, so which mean need to add almost-black into <div class="team-person"> like <div class="team-person almost-black"> right?

@juliaferraioli
Copy link
Member Author

@YC-Kwan that's correct!

@YC-Kwan
Copy link

YC-Kwan commented Apr 25, 2022

Okay, thanks. Is it necessary to install Go and Hugo?

@juliaferraioli
Copy link
Member Author

Generally, I think that's a good thing to do so that you can see how it works before submitting a PR.

@juliaferraioli
Copy link
Member Author

Hi @YC-Kwan, are you still interested in this issue?

@VenkateshThoguta
Copy link

hi,im a beginner tp opensource projects.
i have done bunch of projects
i think i will be able to fix this good first issue
can u please assign me?

@VenkateshThoguta
Copy link

we can add f4 class to div of team-person class
so that we can fix the issue

@juliaferraioli
Copy link
Member Author

Hi @VenkateshThoguta, happy to assign it to you!

@VenkateshThoguta
Copy link

I have made the changes in class in list.html in about section
Made a pull request
Kindly check

@VenkateshThoguta
Copy link

VenkateshThoguta commented Jun 19, 2023 via email

@juliaferraioli
Copy link
Member Author

Hi @VenkateshThoguta, I actually don't see an open PR from you. Can you ensure you submitted it?

@VenkateshThoguta
Copy link

VenkateshThoguta commented Jun 20, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers UX
Projects
None yet
Development

No branches or pull requests

3 participants