-
Notifications
You must be signed in to change notification settings - Fork 608
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
Slightly improve mobile support #167
Comments
No, I'm talking about the UI |
I stilll do not understand your issue. |
Do you mind if I do a UI overhaul & try to make it a little more intuitive or are you attached to the way it works right now? |
In terms of this issue, when the application window is height-limited, many parts of the UI don't account for this. For example, the effects dropdown is very tall and can easily get cut off on small screens so you can't click on the bottom options in the dropdown. |
I looked again, tested and i see and understand these issues:
p.s. i will try to edit this list, so if anything is missed - please add it in comments. |
I am really happy with desktop UI, i believe it works well, but mobile UI can be improved a lot. So please do (you are talking about mobile, right?). |
I'm kind of talking a mixture of everything, though the mobile experience obviously needs the most work.
In terms of the rasterization problem, you could store vector & other tool modification data on the same layer. For example, if I add text, then use the sharpen or blur tool to modify it, the application can store the mouse movements made with the blur or sharpen tool so the rasterized result can be re-created even if the text is changed after the fact. |
Disagree, but I do my best to make it looks as good as possible and try to improve it when i find good ideas, but i am not designer.
It would take some time to implement it, i will skip this for now and focus on more important things.
moved to todo comment #167 (comment) |
I work as a UI designer in my day job. I have ideas on how to improve it, but I don't want to go in and make sweeping changes to the look & feel if you won't accept them. The worst part right now is with the default "dark" theme, the background and text color has too low contrast. There are many other things that can be improved, from how certain controls that are reused throughout the application look and work, to improving layouts. |
All contrast problems should be fixed (added to todo).
Please mention all issues that you see. If it will be not hard to fix it and it will be better for most people, i will agree and implement it. Of course i would like short discussion first to avoid work, that could not be accepted for some reasons. p.s. i had busy day, will check and test your PR tomorrow. |
I think I'm going to take the piecemeal approach and tackle sections of the app one at a time, since there's a ton of functionality in this thing. That's my intent behind #174, just starting somewhere. If we end up replacing an existing pattern in one section of the app with a better one, maybe at that point in time consider applying that pattern across other features. |
One thing specifically for mobile, most mobile browsers trick the website into thinking it has a lower resolution than it actually does, e.g. 1px on the webpage does not equal 1px on the phone screen. This causes noticable aliasing on the canvas, rendered much lower than the phone's screen resolution. Just need to take devicePixelRatio into account when rendering the canvas to fix this. https://developer.mozilla.org/en-US/docs/Web/API/Window/devicePixelRatio |
Hello there.
I like this App so much, and it works on my mobile so well.
The only drawback is that, when content overflows (eg. menu or window content), there's no ability to scroll down to see it.
Keep up the great work!
The text was updated successfully, but these errors were encountered: