Skip to content

Restore Mastodon rel-me verification#13

Merged
MSicc merged 1 commit intomasterfrom
codex/portfolio-modernization
Mar 8, 2026
Merged

Restore Mastodon rel-me verification#13
MSicc merged 1 commit intomasterfrom
codex/portfolio-modernization

Conversation

@MSicc
Copy link
Member

@MSicc MSicc commented Mar 8, 2026

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.

@MSicc MSicc self-assigned this Mar 8, 2026
Copilot AI review requested due to automatic review settings March 8, 2026 19:40
@MSicc MSicc merged commit cb365f6 into master Mar 8, 2026
3 of 4 checks passed
@MSicc MSicc deleted the codex/portfolio-modernization branch March 8, 2026 19:40
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Restores Mastodon “rel=me” verification by conditionally adding rel="me" to the footer contact link for Mastodon.

Changes:

  • Add optional rel_me flag to contact link data.
  • Conditionally prepend me to the rel attribute 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.

Comment on lines +7 to 9
<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>
Copy link

Copilot AI Mar 8, 2026

Choose a reason for hiding this comment

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

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.

Suggested change
<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 %}

Copilot uses AI. Check for mistakes.
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