-
Notifications
You must be signed in to change notification settings - Fork 33
Fix #40: Enhancement of the UI #44
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
base: main
Are you sure you want to change the base?
Conversation
Hey! I'll review the code and let you know by a day :) |
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 @Sambit-Mondal! The code seems to have certain misalignments. Please fix those and I'll merge the PR
@AalokeCode Misalignments as in? Could you kindly elaborate, it'd be very helpful 😄 |
@AalokeCode Hello? |
Hey! Apologies. I've been offline for a bit. The temperature is not centered and the spacing between the date and location is kinda too much. |
@AalokeCode Nvm, I'll fix it by tonight! |
@AalokeCode I've adjusted the UI as per requirements, as shown in the screenshot attached below. Kindly check. |
I've fixed the UI, as per the requirements that I had mentioned in the Issue #40.
Kindly check the below provided video for more details.
Thank you!
Recording.2024-10-26.193652.mp4