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

Add Ethereum to thumbnail copy #89

Merged
merged 2 commits into from
Mar 22, 2024
Merged

Add Ethereum to thumbnail copy #89

merged 2 commits into from
Mar 22, 2024

Conversation

Pabl0cks
Copy link
Member

Description

Add "Ethereum" copy to thumbnail => "Funding meaningful projects across the Ethereum ecosystem"

Related Issues

Closes #88

Copy link

vercel bot commented Mar 22, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
grants-bg ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 22, 2024 10:30am

@carletex
Copy link
Contributor

carletex commented Mar 22, 2024

image

Oh, that p almost touching the E looks a bit weird, right?

Maybe this will fix? (capitalizing rules titles/subtitles are different than in Spanish)

«Titles and Headings: In titles and headings, capitalize the first, last, and all major words (nouns, pronouns, verbs, adjectives, adverbs) but usually not articles, short prepositions, or conjunctions, unless they are the first or last word. This is known as title case.»

image

or more space between the lines?

Not sure, open to suggestions!

@Pabl0cks
Copy link
Member Author

Pabl0cks commented Mar 22, 2024

I've tried some options here:

1 thumbnail1 2 thumbnail2
3 thumbnail3 4 thumbnail4

My personal feedback for each option.

  • 1 looks a bit strange to me, but maybe because I'm not used to it. It's true that in meta-descriptions it's very common to see sentences like that.
  • 2 looks well to me, and capitalize "Projects" may make sense like if we want to emphasize they are not "regular" projects.
  • 3 looks well but is hard to find a word that has all the meaning of "projects". Ideas or builds may not match well for all the stuff we grant.
  • 4 still looks a bit weird to me the separation between the p and E

@carletex
Copy link
Contributor

Thanks Pablo!!

I think I like 2 too.

@Pabl0cks
Copy link
Member Author

I think I like 2 too.

Thx! Just pushed 2 🙌

@carletex carletex merged commit d3221a2 into main Mar 22, 2024
3 checks passed
@carletex carletex deleted the update/thumbnail branch March 22, 2024 11:31
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.

Add "Ethereum" on the thumbnail
2 participants