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

Updated with shadcn UI #4

Merged
merged 34 commits into from
Mar 17, 2024
Merged

Updated with shadcn UI #4

merged 34 commits into from
Mar 17, 2024

Conversation

prokawsar
Copy link
Collaborator

@prokawsar prokawsar commented Mar 5, 2024

Updated deployed version can be seen here
https://ark-gallery.vercel.app/

Changes:

  • Added shadcn as UI library shadcn link
  • Added delete confirmation modal
  • Added zoom slider (will need fix functional behavior)
  • Improved dropdown UI with shadcn

@prokawsar prokawsar self-assigned this Mar 5, 2024
@prokawsar prokawsar requested review from kirillt and Pushkarm029 March 5, 2024 14:33
@kirillt kirillt requested a review from alvinosh March 11, 2024 13:39
Prototype using SvelteKit (#1)
@kirillt
Copy link
Member

kirillt commented Mar 16, 2024

@prokawsar is it possible to make a screenshot or short video demonstrating the change?

I'm just not sure how did it look before..

@kirillt
Copy link
Member

kirillt commented Mar 16, 2024

Also, if we pull commit 265b60a from main branch, then I'll be able to run both build and compare.

@prokawsar
Copy link
Collaborator Author

prokawsar commented Mar 17, 2024

@prokawsar is it possible to make a screenshot or short video demonstrating the change?

I'm just not sure how did it look before..

@kirillt
Old version: https://ark-gallery-9nqnix0t8-prokawsar.vercel.app/
Updated version: https://ark-gallery.vercel.app/
Video with updated tauri build version: https://streamable.com/pn02of

You might need to login into vercel to check old version.

@kirillt
Copy link
Member

kirillt commented Mar 17, 2024

@kirillt kirillt merged commit 2a720a5 into ARK-Builders:main Mar 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants