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
Enhance multi file upload #2912
Conversation
netchampfaris
commented
Mar 22, 2017
•
edited
edited
👍 Nice! Maybe each file button should be on a new line (block), it would be better alignment / consistency. Also maybe the lock can be just normal instead of gold (?) |
frappe/public/js/frappe/upload.js
Outdated
}); | ||
}, | ||
make_file_pill: function(filename, show_private) { | ||
var $file = $(` | ||
<div class="btn-group" role="group" data-filename="${filename}"> |
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.
display='block'
?
Very nice usability help ;-) |
6e62a1d
to
3dd44b3
Compare
Very nice. I like the way that you pay attention to the aesthetics as well as the functionality |
You can now upload files programmatically like ``` frappe.upload.make({ ... files: fileobj_array, ... }) ``` No need to use `multifile_upload` method. Multiple file upload is enabled by default on all Attach dialogs.
3dd44b3
to
9237187
Compare
did you remove the "is" in "is private" ? |