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 list.html #99

Closed

Conversation

VenkateshThoguta
Copy link

Updated class of about section by adding f4

Updated class of about section by adding f4
@VenkateshThoguta VenkateshThoguta requested a review from a team as a code owner June 20, 2023 15:21
@VenkateshThoguta VenkateshThoguta requested review from amcasari and removed request for a team June 20, 2023 15:21
@juliaferraioli juliaferraioli self-requested a review June 20, 2023 16:39
Copy link
Member

@juliaferraioli juliaferraioli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a great start! Can you refer back to this issue comment for other areas that need fixing as well? If you look through the various pages/layouts, you can see similar inconsistencies.

@VenkateshThoguta
Copy link
Author

VenkateshThoguta commented Jun 20, 2023 via email

@juliaferraioli
Copy link
Member

I linked the issue comment that specifies what needs done. You have modified one layout, but others likely need modification as well, including contribute/list.html and posts/list.html.

Copying from that:

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:

opensourcestories.org/layouts/about/list.html

Line 15 in e5d960d

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

opensourcestories.org/layouts/about/list.html

Line 16 in e5d960d

{{ .name }}, {{ .roles }}

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

opensourcestories.org/layouts/about/list.html

Line 17 in e5d960d

{{ .bio }}

@VenkateshThoguta
Copy link
Author

VenkateshThoguta commented Jun 23, 2023 via email

@stale
Copy link

stale bot commented Aug 22, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Aug 22, 2023
@stale stale bot closed this Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants