-
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
Added feature: full screen functionality #6
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Thanks for you contribution! I'll get back to you and I will be adding a prettier config. I see that the PR has updated the single quotes to double quotes. |
@Vidhanvyrs good work and thanks again. Would you be okay to merge the latest main to your branch and run Also, feel free to work on other issue if you'd like :) |
Another thing. Looks like the footer does not auto hide anymore. Please fix that too 🙏🏼 |
Sure why not but I have one question doing so after this do I require to again do a commit and push or what? because see although I know how to solve the issues the main thing that confuses me while doing open source contribution is this like for example if somebody asks u to do some changes in the pr do the pr gets updated or a new pr is added can you please help me understand this flow thing |
Oops I switched it off while testing the button so my fault will roll it back as it was |
I generally keep my feature branch in sync with the branch I am planning to merge into. In this case you should merge main into your branch and resolve all the conflicts. You can do the following:
and it will not show this error if you did it right. Please follow the steps above and push! good luck |
@Vidhanvyrs I have fixed the conflicts for you. Feel free to pull the latest and see if you can fix the issues of footer not auto hiding. Also, if you click escape to exit the fullscreen the icon state does not change. |
@Vidhanvyrs Just checking if you are planning to complete this PR? |
Extremely Sorry @dnafication I am at the hospital now and going to have a medical surgery which is why I was not able to complete the task |
oh! I am sorry to hear that. Please take care of yourself. Don't worry about the task. It can wait. Your health is more important. Good luck with the surgery. I hope everything goes well. Take care! |
hey @dnafication I have successfully implemented the functionality please review and merge it under hacktoberfest if you find it correct else ping me with the changes required
looking forward for the feedback