-
Notifications
You must be signed in to change notification settings - Fork 62
fix journal form #151
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 journal form #151
Conversation
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.
Hi @roshanchatei , why do we need the ./idea folder? If not, please delete it. Apart from that, everything looks fine.
@padmajabhol its because I use IntelliJ IDEA as my text editor, to remove it the maintainer needs to add '/.next/' to .gitignore |
You can do this when you send a commit or add additional commits in this particular PR to do the same. |
Ok |
…racker-frontend into addJournal
@padmajabhol now can you check it out |
@padmajabhol are you good with this now? |
Couldn't check, was away for a few days, doing right away. |
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'm not sure but I'm not able to click on anything on your branch, can you look into this? Let me know if you need help. Sorry for the late review since I was away for a few days.
Ok, I will look into it |
Description
Added bottom margin to the input fields so that when focused on the input field, the bottom border is not being hidden behind the other input field
Fixes #94
Type of change
Checklist: