-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
feat: allow external drop and add dropzone #40674
Conversation
9bfc649
to
8b20faa
Compare
How does it handle rich workspaces? |
This comment was marked as resolved.
This comment was marked as resolved.
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.
Code looks good, but not a fan of how it looks :/
Have we looked at how others are doing it ?
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
8b20faa
to
e7d5d54
Compare
@nimishavijay @jancborchardt please advise updated design in the first comment! |
e7d5d54
to
0735ec5
Compare
Design tuning can/will be a followup though |
Those two colours doesn't really go together? Is this a mistake? :) |
Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
0735ec5
to
35aed73
Compare
Fix #4483
@nextcloud/designers for visual input