Conversation
There was a problem hiding this comment.
Pull request overview
Restores Mastodon “rel=me” verification by conditionally adding rel="me" to the footer contact link for Mastodon.
Changes:
- Add optional
rel_meflag to contact link data. - Conditionally prepend
meto therelattribute for links that request it.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| _includes/site-footer.html | Conditionally adds rel="me" to link markup to support verification. |
| _data/contact.yml | Marks the Mastodon link as rel_me: true to enable verification. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <a href="{{ link.url }}" target="_blank" rel="{% if link.rel_me %}me {% endif %}noopener noreferrer" aria-label="{{ link.label }}" title="{{ link.label }}"> | ||
| <i class="{{ link.icon_class }}" aria-hidden="true"></i> | ||
| </a> |
There was a problem hiding this comment.
rel="me" is intended for identity verification and generally only works reliably when linking from the verifying site to the profile without target="_blank" semantics; some verifiers are strict about the exact link relation and/or the browsing context. Consider only adding target="_blank" (and noopener noreferrer) for non-rel_me links, or make target_blank configurable per-link in the data so the Mastodon link can omit it while preserving the behavior for other links.
| <a href="{{ link.url }}" target="_blank" rel="{% if link.rel_me %}me {% endif %}noopener noreferrer" aria-label="{{ link.label }}" title="{{ link.label }}"> | |
| <i class="{{ link.icon_class }}" aria-hidden="true"></i> | |
| </a> | |
| {% if link.rel_me %} | |
| <a href="{{ link.url }}" rel="me" aria-label="{{ link.label }}" title="{{ link.label }}"> | |
| <i class="{{ link.icon_class }}" aria-hidden="true"></i> | |
| </a> | |
| {% else %} | |
| <a href="{{ link.url }}" target="_blank" rel="noopener noreferrer" aria-label="{{ link.label }}" title="{{ link.label }}"> | |
| <i class="{{ link.icon_class }}" aria-hidden="true"></i> | |
| </a> | |
| {% endif %} |
Summary
Describe the change in a few sentences.
Testing
List the checks you ran or explain what still needs validation.
Context
Link any related issue or add background that helps explain the change.