-
Notifications
You must be signed in to change notification settings - Fork 164
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
improve ui #51
base: main
Are you sure you want to change the base?
improve ui #51
Conversation
@jywarren @gauravano please review |
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.
@VladimirMikulic should we merge this now and continue in a follow-up? Thank you for sharing screenshots, i didn't have a chance to do a close review here. Thank you @avsingh999 for your patience as well!
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.
@jywarren we can merge this now and create an FTO for adding padding to the footer.
OK, i like it! Hmm, though, the conflicts are a bit much for me to go through right now. @avsingh999 if you are still interested, I'd love a bit of help with resolving conflicts? |
issue #7

