-
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
Added comments + Fixed indentations #200
base: main
Are you sure you want to change the base?
Conversation
Thanks for opening this pull request! |
Hi @anushkachauhxn, can you add a link to the issue you're resolving? You can add it in the format |
Hi @TildaDares, this is a simple enhancement to the code. Should I create a new issue for this? |
Never mind, I understand what you did. |
Hi @anushkachauhxn, this PR is on hold because we plan on redesigning the Infragram homepage as an Outreachy project https://publiclab.org/wiki/gsoc-ideas#Outreachy:+Infragram.org+full-screen+UI+design. You can open an issue that lists all of the things you plan to change/redesign along with wireframes/sketches if you want to work on this project. This list/note will serve as your proposal to work on this project. If you don't plan on working on this project, you can make suggestions or discuss improvements you think should be made to the Infragram UI in our project discussion issue publiclab/plots2#10754. We appreciate your contributions to Public Lab. Thank you! |
I'll do that. Thank you so much! |
@TildaDares Can I continue with this pull request as an Outreachy Contributor now? Also, should I open a new issue for this? |
Hi @anushkachauhxn, the UI infragram design project will change a lot of code so if you refactor it now, it’ll be completely lost when the internship starts. |
Added comments + Fixed indentations in index.html file