-
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
Make UI more responsive and CSS cleanup #92
Make UI more responsive and CSS cleanup #92
Conversation
hello @harshkhandeparkar . I'm done with the work. Can I please get a review and merge it? |
@uday-kalyan-s Please rebase the PR. Also, can you send a few screenshots at different screen sizes? |
ec9a54b
to
cc97e8d
Compare
hello @harshkhandeparkar . i have rebased it. can you please review it now |
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 don't like reading CSS. We'll test it in prod.
Looks fine. However, perhaps the TT height can be reduced on mobile. |
uhh how was this closed? I dont have to make a new PR right |
duplicate of #89 (comment) because it got wiped out cleaning commit history