Skip to content
This repository has been archived by the owner on Oct 5, 2024. It is now read-only.

Feature/upload field branch (draft) #116

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

kaffikaffi
Copy link
Contributor

An upload field with support for multiple files and drag & drop.
For now this is a draft, and I want to continue working on it. Functionality with back-end has not yet been added, but will be added later.

@plusk
Copy link
Member

plusk commented Oct 27, 2019

GitHub has a thing called "draft pull requests" which are pretty neato, in the future you can make those instead. You can read more about them and how to make them here. That aside:
Awesome diff. Great size. Look thick. Solid. Tight. Keep us all posted on your continued progress with any new commits or pull requests. Show us what you got man. Wanna see how freakn’ huge, solid, thick and tight your contributions stats can get. Thanks for the motivation
POTENTIALLY sign me the FUCK up 👋👀👋👀👋👀👋👀👋👀 average shit modera̷̶te sHit 👋 thats some ALright 👋👋shit right 👋👋 th 👋 ere 👋👋👋 right 🆗 there 🆗 if i do ƽaү so my self🆗 i say so 🆗 that could be what im talking about right there right there (chorus: ʳᶦᵍʰᵗ ᵗʰᵉʳᵉ) mMMMMᎷМ 🆗 👋 👋👋НO0ОଠOOOOOОଠଠOoooᵒᵒᵒᵒᵒᵒᵒᵒᵒ 👋 👋👋 👋 🆗 👋 👀 👀 👀 👋👋Potentially good shit

Copy link
Member

@niklasmh niklasmh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be nice with some screenshots in the PR (even though it is still a draft). Screenshots are useful for documentation, makes it easy for lazy developers who do not want to checkout into another branch to see the result and it is quite easy to do: ctrl+shift+PrtSc (ubuntu), mark area, paste into a comment/PR description.

Anyway, great work so far!

@FluidSense FluidSense marked this pull request as draft October 19, 2020 10:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants