Problem/Motivation

Mobile YouTube urls (copied from mobile browsers) aren't changed to YouTube video players.

Steps to reproduce

Insert a mobile YouTube link (e.g. https://m.youtube.com/watch?v=vhAAHqk7MHs) into a text field with a text format where URL to Video Filter is enabled.

Proposed resolution

Modify the "www" pattern to also match urls with an m at the www substring's position.

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#6 url_to_video_filter-3328526-2--on-top-of--3191775-6.patch3.41 KBhuzooka
#3 url_to_video_filter--3191775-4--plus--3328526-2.patch3.28 KBhuzooka
#2 url_to_video_filter-mobile_youtube-3328526-2.patch2.52 KBhuzooka

Comments

huzooka created an issue. See original summary.

huzooka’s picture

huzooka
Primary language Hungarian
Location Hungary 🇭🇺🇪🇺
commented
Assigned: huzooka » Unassigned
Status: Active » Needs review
StatusFileSize
new url_to_video_filter-mobile_youtube-3328526-2.patch2.52 KB
huzooka’s picture

huzooka
Primary language Hungarian
Location Hungary 🇭🇺🇪🇺
commented
StatusFileSize
new url_to_video_filter--3191775-4--plus--3328526-2.patch3.28 KB

Patch on top of #3191775-4: URL Replacement Breaks with HTML Element Attributes.

gaurav-mathur’s picture

gaurav-mathur commented
Assigned: Unassigned » gaurav-mathur
gaurav-mathur’s picture

gaurav-mathur commented
Assigned: gaurav-mathur » Unassigned

Verified and tested patch#3 on Drupal 8.x-1.x-dev patch applied successfully.
Thank You.

huzooka’s picture

huzooka
Primary language Hungarian
Location Hungary 🇭🇺🇪🇺
commented
StatusFileSize
new url_to_video_filter-3328526-2--on-top-of--3191775-6.patch3.41 KB

New patch on top of #3191775-6: URL Replacement Breaks with HTML Element Attributes

jaypan’s picture

jaypan
Location Victoria, BC
commented
Status: Needs review » Fixed

Thank you, this has been implemented in 3.x.x.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.