-
Notifications
You must be signed in to change notification settings - Fork 40
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: update design #528
base: main
Are you sure you want to change the base?
feat: update design #528
Conversation
@Morl99 what do you think? |
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.
I like that it looks a lot cleaner and more professional than my original layout.
I do think, that the instructions and the file uploader now take a lot of real estate. On my resolution, that is more than half of the screen height. While some of that is attributed to whitespace (that we could reduce a bit in my opinion), maybe we could move the instructions to somewhere else. Or alternatively, change the design to be more dynamic and collapse once a file is selected (with the option to reopen).
What do you think?
Summarising our phone call: |
@Morl99 I adopted the design and updated screenshots. Please have a look :) |
@Morl99 can you check the updated design? |
No description provided.