-
Notifications
You must be signed in to change notification settings - Fork 6
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
Made the login/signup page responsive #14
base: main
Are you sure you want to change the base?
Made the login/signup page responsive #14
Conversation
@dhanrajchaurasia I have made the PR . Please check if any changes are required . I am not able to submit the PR on contrihub since it is not visible in my profile . |
well you won't be awarded points if you don't see this issue as assigned in your profile page. @shank03 can you fix this problem? |
@dhanrajchaurasia Sir what should I do now ? There could be some glitch . Could you pls assign the issue to me again maybe then it will show on my contrihub profile . |
okay make another request if you can I will accept that |
@dhanrajchaurasia Sir I have requested the issue again on contrihub . Please assign it to me again |
@dhanrajchaurasia Sir I have submitted the PR on contrihub . Please check |
@dhanrajchaurasia Sir I have implemented the feature . Please check if anything else needs to be done . |
@Manan-Arora31 what actually did you do? increased the font size? is it responsive? side images are overflowing! |
@dhanrajchaurasia Sir I have fixed the bugs in the responsive design . Please check now . |
@dhanrajchaurasia Please check if any other changes needs to be done . |
Well @Manan-Arora31 I reviewed the PR, not satisfied with your work (you are doing the same stuff again and again without reading my instructions!)! |
@dhanrajchaurasia Sir I was ready to work on the issue so was constantly asking again and again for any improvements . I changed what I undertood should be done, as mentioned in the change I had made the image responsive . If there were further changes required you could have said so |
Description
Made the login and signup page responsive . And corrected the bugs in the frontend .
Related Issue
Fixes #3 (issue)
Type of change
Motivation and Context
It is required as the webpage is not responsive so won't work in mobile devices.
How Has This Been Tested?
It has been tested by running on my local system.
Screenshots (if appropriate):
Checklist:
Test Configuration: