Skip to content

Conversation

srapilly
Copy link
Contributor

The html lang attribute is present for the title and card provider, but it was not present for the description.

I also saw it was not in the https://docs.joinmastodon.org/entities/PreviewCard/ documentation

@srapilly srapilly changed the title HTMLLang attribute for PreviewCard description HTML lang attribute for PreviewCard description Oct 23, 2023
@srapilly srapilly marked this pull request as ready for review October 23, 2023 00:30
@vmstan vmstan added javascript Pull requests that update Javascript code area/web interface Related to the Mastodon web interface labels Oct 23, 2023
@ClearlyClaire ClearlyClaire changed the title HTML lang attribute for PreviewCard description Add HTML lang attribute to preview card descriptions Oct 23, 2023
@srapilly
Copy link
Contributor Author

@Gargron Hi, thanks for the review. What's the process for merging after it's approved ? I also have this PR #27409 (need to fix the conflict on this one)

@Gargron Gargron added this pull request to the merge queue Oct 26, 2023
Merged via the queue into mastodon:main with commit 2d8f759 Oct 26, 2023
ponapalt pushed a commit to ponapalt/mastodon that referenced this pull request Oct 29, 2023
zunda pushed a commit to zunda/mastodon that referenced this pull request Oct 29, 2023
audiodude pushed a commit to audiodude/mastodon that referenced this pull request Nov 16, 2023
000hen pushed a commit to thenapnetwork/nap-mastodon that referenced this pull request Dec 6, 2023
vmstan pushed a commit to vmstan/mastodon that referenced this pull request Dec 14, 2023
vmstan pushed a commit to vmstan/mastodon that referenced this pull request Jan 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/web interface Related to the Mastodon web interface javascript Pull requests that update Javascript code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants