Add external links class option to Markdown configuration #2717
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.
IMPORTANT: Please do not create a Pull Request adding a new feature without discussing it first.
The place to discuss new features is the forum: https://zola.discourse.group/
If you want to add a new feature, please open a thread there first in the feature requests section.
Sanity check:
Code changes
(Delete or ignore this section for documentation changes)
next
branch?If the change is a new feature or adding to/changing an existing one:
Discussed in https://zola.discourse.group/t/style-internal-and-external-links-differently/1501/6.
This PR adds an optional
external_links_class
configuration option that allows adding a CSS class to all external links.This makes it easier to style external links differently from internal ones, for example to add visual indicators for external links.
Example configuration:
with CSS:
It looks like:
Note: for the docs, I wasn't sure whether to do (same as
author
):external_links_class =
or (make it super obvious it's a string):
Opted for
author
format, for consistency.Happy to make any necessary changes!