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

restructure projects ui #1542

Merged
merged 11 commits into from
Jan 11, 2025
Merged

restructure projects ui #1542

merged 11 commits into from
Jan 11, 2025

Conversation

raspberri05
Copy link
Contributor

Worked with @evanugarte to remake the projects ui with each project card being in a separate row and the icons being removed.
please leave any suggestions below.
also lmk if i should add the more recent projects (issue #1525)

image

@raspberri05 raspberri05 added the Front End Things that make changes in the src/ directory, excluding src/APIFunctions/. label Oct 15, 2024
@raspberri05 raspberri05 self-assigned this Oct 15, 2024
@raspberri05 raspberri05 requested review from evanugarte, steeevin88 and joshua-demo and removed request for steeevin88 October 15, 2024 06:04
@steeevin88
Copy link
Collaborator

agree with josh's comments, if you could add SCEta to the page it would be really cool 😄

i personally dislike having the pancake printer + RFID projects because they were never finished, I added them ~2 years ago when we didn't have that many other projects. maybe we could add them? (ex. Cleezy, Messaging)

@weslayer
Copy link
Contributor

nice work bro, it would be very cool to add recent projects 😎😎

@raspberri05
Copy link
Contributor Author

agree with josh's comments, if you could add SCEta to the page it would be really cool 😄

i personally dislike having the pancake printer + RFID projects because they were never finished, I added them ~2 years ago when we didn't have that many other projects. maybe we could add them? (ex. Cleezy, Messaging)

Ok, I can remove the pancake printer and RFID projects and add the newer completed ones

@raspberri05
Copy link
Contributor Author

@joshua-demo @steeevin88 I fixed the layout and padding for mobile and added the new projects. Wasn't entirely sure about the details of the new projects added, so suggestions of what to put would helpful. Also there is a placeholder image for Cleezy bc i cant access it to take a screenshot.
Screenshot 2024-10-18 123535

Screenshot 2024-10-18 123553

@steeevin88
Copy link
Collaborator

steeevin88 commented Oct 22, 2024

https://github.com/user-attachments/assets/de1017ac-ca79-46e5-b3b4-5e62666713fb

here's a picture for Cleezy

everything else LGTM, nice work 😃

src/Pages/Projects/Components/ProjectCard.js Outdated Show resolved Hide resolved
src/Pages/Projects/Projects.js Outdated Show resolved Hide resolved
@raspberri05 raspberri05 removed the request for review from joshua-demo October 23, 2024 21:00
Co-authored-by: Evan Ugarte <36345325+evanugarte@users.noreply.github.com>
raspberri05 and others added 2 commits October 23, 2024 18:16
Co-authored-by: Evan Ugarte <36345325+evanugarte@users.noreply.github.com>
@evanugarte evanugarte merged commit 5f89a18 into dev Jan 11, 2025
4 checks passed
@evanugarte evanugarte deleted the restructure-projects-ui branch January 11, 2025 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Front End Things that make changes in the src/ directory, excluding src/APIFunctions/.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants