Przeglądaj źródła

Merge pull request #83 from ckeditor/t/82

Feature: Added support for mobile YouTube URLs in the default providers configuration. Closes #82.
Aleksander Nowodzinski 6 lat temu
rodzic
commit
d6920b4dba

+ 2 - 2
packages/ckeditor5-media-embed/src/mediaembedediting.js

@@ -78,8 +78,8 @@ export default class MediaEmbedEditing extends Plugin {
 				{
 					name: 'youtube',
 					url: [
-						/^youtube\.com\/watch\?v=([\w-]+)/,
-						/^youtube\.com\/v\/([\w-]+)/,
+						/^(?:m\.)?youtube\.com\/watch\?v=([\w-]+)/,
+						/^(?:m\.)?youtube\.com\/v\/([\w-]+)/,
 						/^youtube\.com\/embed\/([\w-]+)/,
 						/^youtu\.be\/([\w-]+)/
 					],

+ 4 - 0
packages/ckeditor5-media-embed/tests/mediaembedediting.js

@@ -206,10 +206,14 @@ describe( 'MediaEmbedEditing', () => {
 								'https://www.youtube.com/watch?v=foo',
 								'www.youtube.com/watch?v=foo',
 								'youtube.com/watch?v=foo',
+								'https://m.youtube.com/watch?v=foo',
+								'm.youtube.com/watch?v=foo',
 
 								'https://www.youtube.com/v/foo',
 								'www.youtube.com/v/foo',
 								'youtube.com/v/foo',
+								'https://m.youtube.com/v/foo',
+								'm.youtube.com/v/foo',
 
 								'https://www.youtube.com/embed/foo',
 								'www.youtube.com/embed/foo',