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

Upload div elements changed to <input/> type of 'file' elements #11 #18

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

Conversation

Halim-Shams
Copy link

Hey i just changed the divs to type of 'file'

Halim-Shams and others added 3 commits December 25, 2022 07:06
Upload div elements changed to <input/> type of 'file' elements
@vercel
Copy link

vercel bot commented Dec 25, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
comicfun ✅ Ready (Inspect) Visit Preview Dec 25, 2022 at 11:27AM (UTC)

@brandonbyr4
Copy link
Owner

Looks good, but let me preview this one once I am at my PC.

@brandonbyr4
Copy link
Owner

@Halim-Shams any way the inputs could be visible, instead of hidden? And replce the divs?

@brandonbyr4
Copy link
Owner

Nevermind, I see what you were doing. hidden inputs with a for attribute. I included these changes in the newest PR.

@Halim-Shams
Copy link
Author

Hey @brandonbyr4 was somthing wrong with my pr that you closed it?

@brandonbyr4 brandonbyr4 reopened this Dec 27, 2022
@brandonbyr4
Copy link
Owner

I thought it got squashed into the next merge, but looks like there was conflicts. Check the conflicts.

@Halim-Shams
Copy link
Author

@brandonbyr4 did you run it locally to see where is the problem? I run it locally and there was no problem at all.

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.

None yet

2 participants