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

Make R Consortium more prominent on home page of website #123

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

jcabercrombie
Copy link
Contributor

Hi @joseph-rickert,

To make the R Consortium pieces on our home page more prominent, I added/changed the following:

  1. Made the R Consortium a part of the image used in the very first carousel item on our home page.
  2. Added text for and a link to the R Consortium in the text of the first carousel item on our home page.
  3. Redid the logo in the "Partner Initiatives" section we have toward the bottom of the home page so that it fits with others. It seems to only work with more square logos.
  4. I made the R Consortium appear first in the list of partner initiatives to prioritize it there as well.

After checking out the deployed sample site in this PR, you let me know if this is more along the lines of what you were hoping to have? @anujadas185 relayed to me a quick message this morning about your request to make things more prominent and these were the changes I thought could work best.

Note: changing the actual top left logo on the site is not possible with our current theme because it is restricted to a really small amount of pixels, and unfortunately any text and a smaller R Consortium logo become too blurred/pixelated to be readable. I tried a bunch of different versions and nothing was working out.

Thank you,
Jaxon

@joseph-rickert
Copy link

joseph-rickert commented Aug 14, 2024 via email

@jcabercrombie
Copy link
Contributor Author

@joseph-rickert - sounds great! Thank you for reaching out to other folks so we can display things best. Looking forward to what others say and what we end up doing!

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