kivy-garden / kivy-garden.github.io

Add your membership requests here
https://kivy-garden.github.io
MIT License
23 stars 9 forks source link

Please add facelock to kivy-garden #131

Closed yaki29 closed 7 years ago

yaki29 commented 7 years ago

I want to add a flower named FaceLock to kivy-garden. Please add garden.facelock to kivy-garden.

KeyWeeUsr commented 7 years ago

I'm not sure if this is the flower (you didn't provide link here), so I guess that's the thing as I don't see a similar thing in here.

I have some requests, because the repo seems to be in a seriously bad state:

And basically the repo is over 33MB (imagine someone installing it). It'd be probably the best to create a new repository e.g. same-named as the flower you want to add (garden.facelock) and include only code + really necessary stuff (readme, screenshot).

If the widget requires that amount of files for whatever reason (I don't believe that), it would be nice to warn the user in the readme and include only the downloader.

Anyway, any widget that depends on internet connection won't be very happy widget, unless it's explicitly a widget that works with internet/networking.

yaki29 commented 7 years ago

@KeyWeeUsr Sorry for that, I wasn't aware that we are supposed to add a link to our flower as I saw some of the earlier request they didn't mention the links. I have made a new repository for that and removed all the clumsiness as that was just for my use. The final link to my flower is https://github.com/yaki29/facelock

KeyWeeUsr commented 7 years ago

Well, I think those were just people asking for membership, but I'm kind of glad I poked more here 😋

Anyway, still some issues persist:

After that you may proceed with the transfering the repo e.g. to me and I'll put it to the garden with you ^^

yaki29 commented 7 years ago

@KeyWeeUsr I have corrected the things that you mentioned. For the second statement, I have already taken care about that, any person with two eyes, a nose and a lip pair can unlock it :yum: . That cascade detects general frontal faces.