-
Notifications
You must be signed in to change notification settings - Fork 2k
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
comments on nice v0.7 modal #99
Comments
Thank you! Still WIP, some of the issues you mentioned are part of the work for the next release. |
Let’s try to keep UI design discussion in #96 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Wow what a huge step - looks pretty nice!
Many thanks for all your work in this project!
As always some comments on the actual state of example "Modal":
& 2) there are some unessarry scrollbars visible (seen in FF and opera)
if filetype is unknown a question mark is shown: may be alittle confusing/may look broken to the user.
A possible solution may be simply showing the file extenstion instead.
long file names were cut (looks broken to user)
hmm white progressbar (0%) may look broken to the user or like a hidden element..
looks like something (a word) is missing (way to symbols at the bottom is long...)
cloud symbol is nice but not every body (user) likes the cloud and some are also afraid of it...
Loading files to a server of the company they trust is no problem for them at all :-)
-> maybe on could make it easily possible to show e.g. websites favicon (highres) or something different...
The text was updated successfully, but these errors were encountered: