Skip to content

Add pygments CSS block in browsable API views #5584

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

Conversation

merwok
Copy link
Contributor

@merwok merwok commented Nov 9, 2017

Description

PRs #5462 and #5513 added a pygments plugin to the markdown converter for API docs. The same conversion is applied by DocumentationRenderer and BrowsableAPIRenderer, but the latter was missing the style block with the pygments CSS declarations.

This PR adds it.

@carltongibson carltongibson added this to the v3.7.4 milestone Nov 10, 2017
Copy link
Collaborator

@carltongibson carltongibson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. Thanks!

@carltongibson carltongibson merged commit 57ad208 into encode:master Nov 10, 2017
@merwok merwok deleted the feature/highlight-code-blocks-in-browsable-api branch November 10, 2017 14:58
@merwok
Copy link
Contributor Author

merwok commented Nov 10, 2017

Oops I think I forgot to include a template change :(

merwok added a commit to merwok-forks/django-rest-framework that referenced this pull request Nov 10, 2017
merwok added a commit to merwok-forks/django-rest-framework that referenced this pull request Nov 10, 2017
@merwok merwok mentioned this pull request Nov 10, 2017
carltongibson pushed a commit that referenced this pull request Nov 11, 2017
pchiquet pushed a commit to pchiquet/django-rest-framework that referenced this pull request Nov 17, 2020
pchiquet pushed a commit to pchiquet/django-rest-framework that referenced this pull request Nov 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants