Detect hashtag using rel instead of class #2175

Closed
tae wants to merge 0 commits from gitlab-mr-iid-908 into develop
Member

closes #293

closes #293
Member

have we been setting the link rel for a long time? might be safer to just do an OR between the two

have we been setting the link rel for a long time? might be safer to just do an OR between the two
Author
Member

updated.

updated.
Author
Member

@shpuld updated

@shpuld updated
Member

According to https://www.w3.org/TR/html401/struct/links.html#adef-rel

The value of this attribute is a space-separated list of link types.

So it should probably be target.rel.match(/(?:^|\s)tag(?:$|\s)/)

According to https://www.w3.org/TR/html401/struct/links.html#adef-rel > The value of this attribute is a space-separated list of link types. So it should probably be `target.rel.match(/(?:^|\s)tag(?:$|\s)/)`
Author
Member

@rinpatch Thank you! I didn't know it accepts a list. 😀

@rinpatch Thank you! I didn't know it accepts a list. :grinning:
Author
Member

@rinpatch updated.

@rinpatch updated.

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
pleroma/pleroma-fe!2175
No description provided.