-
Notifications
You must be signed in to change notification settings - Fork 345
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
Landing Page - Make Requested Changes #1473
Conversation
Added translate markups to strings. Delete commented code in landing.css Ran landing.css through a CSS beautifier.
Fixed issue with translate not being recognized loading i18n
Fixed translate for Developers near end of file.
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.
This looks good so far. Would you upload a screenshot of the new front page rendered in your local development environment?
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.
This looks good. Please make sure to resolve the conflicts so we can merge this PR.
Conflicts resolved. |
There are too many results to compare View more on Code Climate. |
@all-contributors please add @rdhoge for code |
I've put up a pull request to add @rdhoge! 🎉 |
@all-contributors please add @ApoorvaKashyap for code |
@ApoorvaKashyap already contributed before to code |
Closes #1420
Description
Addresses requests to add translate tags and clean up CSS code for pull request #1420.