-
Notifications
You must be signed in to change notification settings - Fork 14
Implement "suggestions, problems, feedback" GitHub footer #43
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
Comments
This was working in nanoc using the article's slug, etc, and we should be able to something similar in the WordPress theme |
Working on this right now. |
Pull request made on web-base-template jquery/jquery-wp-content#72 |
I want to have that for pretty much every site we have. So far jQuery UI and QUnit have live sites using web-base-template. Can we generalize the solution above to those, and beyond? |
@jzaefferer It is already generalized. If you take a look at the code, it will work from any theme, not just learn.jquery.com. |
This issue will be fixed when this PR is merged in web-base-template jquery/jquery-wp-content#72 |
Landed - Closing |
Each article's footer should include a link to the same article in the learn.jquery.com master on GitHub. Though the link obviously won't work for new articles in development, it will be just fine in production on learn.jquery.com, which is where it counts here.
The text was updated successfully, but these errors were encountered: