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

Created Borders and Aligned Recruitments Page #39

Merged
merged 8 commits into from
Nov 6, 2023

Conversation

amaryllis-sun
Copy link
Contributor

Since all the information for the departments was directly next to each other, I added borders around each department along with some padding to help viewers distinguish the information as well as make it easier to digest. Furthermore, I just added some smaller styling stuff (e.g. colors and rounded borders) to make it more visually appealing and stand out on the website.

@leomet07
Copy link
Member

leomet07 commented Nov 6, 2023

Hey Amaryllis (@amaryllis-sun )!

Nice job on making this pull request! I really liked the addition of the borders and raising the line spacing.

My only comments is to make sure to use the accent color (with var(--accent) in the css and use the rem unit (aka the body text size, which is 16px in this case) for consistency! The purple color was beautiful but it didn't have much contrast on dark mode, which isn't great for the accessibility of the spec site!

Other than that, thanks for your work! Check out your contributions on stuyspec.com, they're already live!

Make sure to rebase and maintain your fork now! (See the "Next Steps" section at the end of the instructions sent out!)

@leomet07 leomet07 merged commit 6867670 into stuyspec:main Nov 6, 2023
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