-
Notifications
You must be signed in to change notification settings - Fork 57
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
Fix: UI improvements in navbar, status, compare page. #548
Conversation
hey @frouioui ! Can you share your views on this pls and guide me further. (I'm quite new to open source) |
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.
Hey @whatDeepak, can you check out the file comments? Thanks!
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.
Left a review that should be addressed, otherwise this is a great contribution, thanks a bunch 🙇🏻
<div className="flex flex-col md:flex-row gap-4 overflow-hidden"> | ||
<ComparisonInput | ||
name="old" | ||
className="rounded-l-full" | ||
className="text-xl px-6 py-4 bg-background border-2 border-gray-300 rounded-full focus:border-primary focus:outline-none" | ||
setGitRef={setGitRef} | ||
gitRef={gitRef} | ||
/> | ||
<ComparisonInput | ||
name="new" | ||
className="rounded-r-full " | ||
className="text-xl px-6 py-4 bg-background border-2 border-gray-300 rounded-full focus:border-primary focus:outline-none" |
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.
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.
got it, I'll make the input style consistent across pages
what i suggest: instead of keeping separate input boxes, i'll follow the old style only and make that responsive by decreasing the size of that in small devices
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.
hey @frouioui , kindly check if this is fine by you and addresses the review comments correctly?
this fix brings responsiveness to compare page while keeping the input style consistent with older design.
Signed-off-by: Deepak <[email protected]>
Signed-off-by: Deepak <[email protected]>
Hello again @whatDeepak, truly thank you for taking the time to work on this. @Jad31 recently revamped these pages/components, I will then close this PR for now. |
This PR enhances the responsiveness and user experience of the application.
Issue Number:
Screenshots/videos:
Before:
After:
I believe these changes will significantly enhance the overall usability and accessibility of the application, contributing to a more positive user experience.
Additionally, I'd like to request access to secret key necessary to further contribute to the project. Having access to these resources will enable me to continue making meaningful contributions to the project's development.
I'm looking forward to your feedback and suggestions for further improvements. :)