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

explore standard multi-source upload with Uppy package #212

Closed
jywarren opened this issue Apr 15, 2018 · 21 comments
Closed

explore standard multi-source upload with Uppy package #212

jywarren opened this issue Apr 15, 2018 · 21 comments
Labels
has-pull-request Issues which have a PR open new-feature

Comments

@jywarren
Copy link
Member

jywarren commented Apr 15, 2018

https://www.npmjs.com/package/uppy

Demo: https://uppy.io/examples/dashboard/

image

@jywarren
Copy link
Member Author

GitMate.io thinks the contributor most likely able to help you is @ccpandhare.

@jywarren
Copy link
Member Author

We could likely integrate this into the "load-image" starter-module -- https://github.com/publiclab/image-sequencer/blob/master/src/LoadImage.js

We might actually merge the "drop zone" in demo.js with the load-image module, because it could show an example image but also say drag in a new image to begin.

@tech4GT
Copy link
Member

tech4GT commented Apr 28, 2018

@jywarren that makes a lot of sense!! Great find!!

@harshithpabbati
Copy link

Hello
Is anyone working on it or can i give it a shot???

@harshithpabbati
Copy link

Do i directly integrate uppy into this ??

@harshithpabbati
Copy link

@publiclab/is-reviewers

@jywarren
Copy link
Member Author

jywarren commented Feb 28, 2019 via email

@harshithpabbati
Copy link

uppy
@jywarren Was this ok????

@jywarren
Copy link
Member Author

jywarren commented Mar 3, 2019 via email

@harshithpabbati
Copy link

Yeah i will make it to access the webcam and share you the gif.

@harshithpabbati
Copy link

Oh wow, doesn't it allow for a few other options like webcam access? Can you show a gif? And, would you mind making the area less tall, like maybe half the height? Thanks!!

On Sun, Mar 3, 2019, 10:23 AM Harshith pabbati @.***> wrote: [image: uppy] https://user-images.githubusercontent.com/43822585/53697312-4b888c80-3df6-11e9-901b-a490df141075.png @jywarren https://github.com/jywarren Was this ok???? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#212 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AABfJ3bx9aNQygqbxNwbrBmAFSzk1X4bks5vS-jbgaJpZM4TVqzw .

@jywarren I am unable to make it small as I am using the plugins.

@harshkhandeparkar
Copy link
Member

Harshith you can check the class that has been given to the drag and drop area and you give it some custom CSS with an !important to style it differently.

@harshithpabbati
Copy link

No that's not the problem with css in image-sequencer it's with the plugins of uppy

@harshkhandeparkar
Copy link
Member

harshkhandeparkar commented Mar 4, 2019

The uppy plugin adds the drag and drop box right? And warren(even me) want to make ot smaller in height?

@harshithpabbati
Copy link

image
Ha done with it.

@harshithpabbati
Copy link

harshithpabbati commented Mar 4, 2019

Is this ok ??? Do we need the dotted lines around it??

@harshkhandeparkar
Copy link
Member

harshkhandeparkar commented Mar 4, 2019

That looks better but I think it is not very obvious to the eye. It should stand out a bit more. You can add the dotted border or alternatively thicken the border of the box itself.

@harshithpabbati
Copy link

image
checkout this !! Harsh

@harshkhandeparkar
Copy link
Member

harshkhandeparkar commented Mar 4, 2019

Awesome!! Are you going to open a PR? If yes then please add the has-pull-request label to the issue and new-feature and user-interface to the PR. You can do that to all new PRs. That is a new practice which proposed recently. Thanks.

@harshithpabbati
Copy link

ok

@harshkhandeparkar
Copy link
Member

We'll find a new solution while developing the new UI. This one isn't working properly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
has-pull-request Issues which have a PR open new-feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants