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

Gallery image upload #56

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

zoranmesec
Copy link

Created image upload functionality in crag gallery:

  • new image upload component (for logged in users only)
  • added image upload component to crag gallery page
  • progress bar standalone component
  • sandbox page for progress bar

src/components/image-upload/image-upload.tsx Outdated Show resolved Hide resolved
src/components/image-upload/image-upload.tsx Show resolved Hide resolved
src/components/image-upload/image-upload.tsx Outdated Show resolved Hide resolved
src/components/image-upload/image-upload.tsx Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
src/components/image-upload/image-upload.tsx Outdated Show resolved Hide resolved
@salamca
Copy link
Contributor

salamca commented Nov 16, 2024

maybe have one state variable for each field, and construct form data only before posting to api?

@salamca
Copy link
Contributor

salamca commented Nov 17, 2024

multiple progress bar sizes will probably not be needed. I suggest we keep only one that is designed in figma to avoid confusion and possible misuse in the future. if a need for another size arises will be designed later.

  • font-size iz to large
  • progress bar and percentage text is not vertically aligned

from figma:
image

@zoranmesec
Copy link
Author

#56 (comment) @salamca Progress bar font size and padding fixed.

@zoranmesec zoranmesec requested a review from demshy November 19, 2024 18:18
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.

3 participants