-
Notifications
You must be signed in to change notification settings - Fork 12
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
feat(openchallenges): update team page #2248
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Order the members as follow (roughly by start date; we can revise this order in 6 mo)
- Thomas, Verena, Rong, Maria, Gaia, Jake
- We may end up adding Sage logo to the navbar, in which case adding Sage logo below the team members would be preferable to put some distance between the two logos.
- What is the motivation for removing the team description?
It's a fun description, but I don't know if it adds enough... value? I think if we were to display the Sage logo, we should ideally mention that instead. |
The pics on the team pages looks pixelated. An additional task for this PR is to replace them with high-res versions. Could this be the result of Thumbor's processing? |
Oh, I didn't like how it looked when they were the same size, which is why I changed it. But can switch back if that is the majority opinion. |
Fixes #2197
Changelog
Note
Given the recent change to the CI workflow (and increased cost), the commits for adding the kudos and Sage logo has not yet been pushed. Once a design is approved, I will push the missing commits, then merge.
Preview
Option 1
Sage logo at the top, under the team description.
Option 2
Remove team description, Sage logo at the bottom (under the kudos)