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

Review homepage #622

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Review homepage #622

wants to merge 6 commits into from

Conversation

maelle
Copy link
Member

@maelle maelle commented Aug 25, 2023

Fix #573

Some personalized questions but more feedback is welcome. For instance maybe the packages and community section should not be in the random order bunch.

@mpadge I simply tweaked the software review fragment, would that be enough? Can we tweak it more so that both systems are highlighted? Ideas?

@yabellini

  • what picture could I use for the champions program?
  • could you suggest more text?
  • do you like the current boxes? idea for alternative content (in the three boxes or instead of the boxes)

@jeroen

  • what picture could I use for r-universe? could you add it to this PR?
  • could you suggest more text?
  • do you like the current boxes? idea for alternative content (in the three boxes or instead of the boxes)

@maelle
Copy link
Member Author

maelle commented Sep 1, 2023

Friendly reminder @yabellini @jeroen @mpadge, for when y'all have time 😸

@yabellini
Copy link
Member

Hey, thanks for working on this. I have some comments, maybe it would be good to discuss some at the team meeting.

The champions program part is nice and I like the topic you choose to link to.

@karthik
Copy link
Member

karthik commented Nov 27, 2023

What is the source of these stock images? Why is R-universe a rocket (what does it have to do with rockets?)
Is there a higher-quality image available? Right now it looks quite low resolution and stands out quite a bit.

@maelle
Copy link
Member Author

maelle commented Nov 27, 2023

The pictures are place-holders 😸

@jeroen jeroen force-pushed the main branch 2 times, most recently from d6537da to bee39f9 Compare January 14, 2024 12:21
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.

review homepage a bit?
3 participants