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

Mentor Page #27

Merged
merged 8 commits into from
Apr 9, 2021
Merged

Mentor Page #27

merged 8 commits into from
Apr 9, 2021

Conversation

Aitori
Copy link

@Aitori Aitori commented Mar 22, 2021

Mentor page pull request.

At the time of pull request, the mentor page desktop version is just about done. Some dummy data is in the code so definitely don't merge. Somethings were a little hacky to get in position like the trees and the responsiveness is also finnicky.

What's left to do?
Mobile Version
NavBar -- But the navbar is universal between all the other pages so if one is made you can just plop it in here, or I can tackle that if you want.

I omitted a few stars and don't think I've added in the fireflies yet. I wasn't sure how to handle really tall screens or really wide screens. (I was lazy and copied the entire ground SVG which works well for reasonable screen ratios, but breaks for extreme ones)

@abhinavpappu
Copy link
Member

Awesome, looking great so far!! Yeah don't worry about the NavBar, I'll work on that

@abhinavpappu abhinavpappu changed the title [WIP] Mentor Page Mentor Page Apr 9, 2021
@abhinavpappu abhinavpappu merged commit 4a2297f into dev Apr 9, 2021
@abhinavpappu abhinavpappu deleted the aitori/mentors-page branch April 9, 2021 14:38
abhinavpappu added a commit that referenced this pull request Apr 9, 2021
* mentor page initial

* finish the mentor page thing

* minor edits

* Add mentor data + minor style changes

* Add mentors to nav bar + style updates

* Add fireflies

* Remove unnecessary import

Co-authored-by: Abhinav Pappu <[email protected]>
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