Refactor external link styling to use external_links_class
#2801
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.
Since this is a small refactor, I did not discuss it first. Feel free to close this if it's not wanted!
Code changes
External link styling added in #2522 was implemented before the PR that added external link classes (#2717).
This PR follows up on both by refactoring the config so as to remove the
external_links_no_referrer = true
line, as it was just added as a workaround to style external links.The styling is now done through the
external
class, by settingexternal_links_class = "external"
.The site looks identical, before and after this change.
next
branch?Since it's tiny and only affects the docs/site, I'm targeting
master
. Let me know if you'd prefer this innext
.