Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrap URL links into <a> elements in YouTube descriptions #2679

Closed
dbelokon opened this issue Jan 19, 2022 · 1 comment · Fixed by #2706
Closed

Wrap URL links into <a> elements in YouTube descriptions #2679

dbelokon opened this issue Jan 19, 2022 · 1 comment · Fixed by #2706
Assignees
Labels
area: accessibility Making Telescope accessible for all kinds of users area: front-end type: enhancement New feature or request type: nice to have Feature that'd be nice to have, but not a priority

Comments

@dbelokon
Copy link
Contributor

What would you like to be added

I would like any links found in YouTube descriptions to be actually clickable links, which can be done by being wrapped in an anchor (<a>) element.

Why would you like this to be added

To improve overall navigation. A mobile user might find cumbersome to copy the whole link and paste it in their browser to be able to navigate to that link. The same applies to PC users.

Additional information

In #2581, we added support to track video posts from an RSS feed produced by an old YouTube API. The description of the video is considered the text of a regular blogpost.

In #2596, these video posts are now displayed in the timeline. During review, @humphd noticed that links found in the descriptions do not actually work. This issue is supposed to address this.

@dbelokon dbelokon added type: enhancement New feature or request Priority: Low type: nice to have Feature that'd be nice to have, but not a priority labels Jan 19, 2022
@humphd humphd added area: accessibility Making Telescope accessible for all kinds of users area: front-end labels Jan 19, 2022
@tpmai22
Copy link
Contributor

tpmai22 commented Jan 19, 2022

@dbelokon I think it will be a quick fix. I can work on this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: accessibility Making Telescope accessible for all kinds of users area: front-end type: enhancement New feature or request type: nice to have Feature that'd be nice to have, but not a priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants