-
Notifications
You must be signed in to change notification settings - Fork 144
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
New upload form field #5951
New upload form field #5951
Conversation
Check whether only images can be uploaded
register the language phrase `wcf.global.button.replace`
…allows the call `....click()` in the case of a `click` event.
Add image viewer support
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A good start, but I would like to see the logic between attachments and the upload form field to be deduplicated. Do not use inheritance, but instead check if you can provide some general purpose functions that offload the work. Especially the DOM and upload progress are pretty much set in stone anyway and can easily be handled by a common function.
…e file upload and also `undefined` if no file is uploaded
… for `WoltlabCoreFileElement`.
Remove unnecessary comments
…on and doesn't insert it
…s clicked, to ignore the upload limit
Use a Promise to recognize the beginning of the upload to replace the context
ts/WoltLabSuite/Core/Form/Builder/Field/Controller/FileProcessor.ts
Outdated
Show resolved
Hide resolved
wcfsetup/install/files/lib/system/event/listener/PreloadPhrasesCollectingListener.class.php
Outdated
Show resolved
Hide resolved
wcfsetup/install/files/lib/system/file/processor/FileProcessor.class.php
Outdated
Show resolved
Hide resolved
Co-authored-by: Alexander Ebert <ebert@woltlab.com>
…Component/File/Helper`
Adds a new upload form field with the new upload pipeline (#5668)