Skip to content

prev hackathon carousel#13

Open
Roshanjahan19 wants to merge 1 commit intomainfrom
prev-hackathon-section
Open

prev hackathon carousel#13
Roshanjahan19 wants to merge 1 commit intomainfrom
prev-hackathon-section

Conversation

@Roshanjahan19
Copy link
Copy Markdown
Collaborator

No description provided.

@Roshanjahan19 Roshanjahan19 requested a review from r0m3c as a code owner November 4, 2024 01:37
Copy link
Copy Markdown
Collaborator

@r0m3c r0m3c left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey Michael, awesome work with the code and with finding the libraries. Below are some change suggestions:

  1. Do you see this on your end as well? Not sure if it's only me, but ideally we'd want two rows of pictures (1 column), similar to the figma. Also, if possible, the next image(s) that are next in line when a student clicks the arrow, should kind of show a blurred version of them (similar to the screenshot in the third section)
Screenshot 2024-11-03 at 10 20 38 PM Screenshot 2024-11-03 at 10 22 26 PM Screenshot 2024-11-03 at 10 24 11 PM

Great work, and let me know if I you need help in anything.

  • Leandro G.

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