Separate font size for lstinline
and lstlisting
#229
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Thanks for this great template!
I often find myself needing to change the
code-block-font-size
to a smaller font in order to avoid line wraps in listings. However this also affects inline code. So I've been having to avoid using this variable and instead opt to something like this to avoid the issue:This PR is an example of how to avoid this by separating the font size used in
lstlisting
environments and other listing environments (i.e. inline).Let me know if I should instead fall back to
code-block-font-size
for inline listings instead if you want it to be backwards compatible. (although I prefer defaulting to the surrounding text, as the current implementation does).I will update the documentation if this implementation is approved