-
Notifications
You must be signed in to change notification settings - Fork 0
Added Catalogue Browser UI #6
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
Conversation
Sync-up PR
Prakhar896
left a comment
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.
PR comments made in-person. Great work overall!!
Prakhar896
left a comment
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.
LGTM. Some things have been commented out temporarily as some other critical work down the line awaits this PR.
What was done in this PR
Catalogueat/gallery/catalogue, which consists of the following components:MMSection: A section to render MM images and dataSection: Renders other artefact imagescardItem: Individual artefact items, caters to bothMMSectionandSectioncatalogueItemView: Custom made modal which opens up, showing the details of the cardItemPages are responsive
Added animations for
cardItemandcatalogueItemViewusing framer motionThings to note