-
Notifications
You must be signed in to change notification settings - Fork 26
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
modify the mobile UI for WIMP #90
Comments
Initially the project was made keeping by minimal frontend and not being focused on design and responsiveness. So frontend needs some changes as you mentioned |
As for the layout and design, you will need to try out some things and then see which is better. So you can go ahead with your proposed changes and try some things out like maybe changing margins, padding, responsive font sizes, etc. |
@harshkhandeparkar what are your opinions on this? |
cc: @rohan-b-84 |
Ah I missed this issue. I commented on the PR first. @uday-kalyan-s if you have any proposed solution, you can send screenshots and we can discuss here or on Slack #projects. |
well my issue mostly revolved around fixing the bugs that were observed on Firefox. now that we realised it's a bigger issue on chrome, I might have to modify the issue again. But like you mentioned in slack, idk how to debug it exactly. I'm not sure about the difference taken in chrome and Firefox here. ig we can discuss it in slack? |
We can resolve the Chrome issue separately. Let's discuss that on Slack. For now, you can work with devtools responsive mode or on Firefox mobile. |
discussions
evident bugs
However i believe a varying size input may be better for smaller names (see "Arun Ghosh") keeping a min if 45% and a max of 90%
splitting this such that 20% of the space goes to the dept code might fix the issue
suggestions
this is hardly even visible but removing it might make the page seem really dull and text-ish. I honestly don't know how to accommodate for it.
tasks
Also I would love to take up the issue myself. I will make a proper PR for it soon
requesting for disc @Prasanthi-Peram @Dami-18
The text was updated successfully, but these errors were encountered: