Skip to content
This repository has been archived by the owner on Jun 26, 2020. It is now read-only.

Commit

Permalink
Merge f3c2e41 into 3fcc3f9
Browse files Browse the repository at this point in the history
  • Loading branch information
msamsel committed May 23, 2019
2 parents 3fcc3f9 + f3c2e41 commit f012769
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 2 deletions.
3 changes: 2 additions & 1 deletion src/mediaregistry.js
Expand Up @@ -298,7 +298,8 @@ class Media {
tag: 'a',
attributes: {
class: 'ck-media__placeholder__url',
target: 'new',
target: '_blank',
rel: 'noopener noreferrer',
href: this.url
},
children: [
Expand Down
2 changes: 1 addition & 1 deletion tests/mediaembedediting.js
Expand Up @@ -927,7 +927,7 @@ describe( 'MediaEmbedEditing', () => {
'<div[^>]+>' +
'<div class="ck ck-media__placeholder ck-reset_all">' +
'<div class="ck-media__placeholder__icon">.*</div>' +
`<a class="ck-media__placeholder__url" href="${ expectedUrl }" target="new">` +
`<a class="ck-media__placeholder__url" href="${ expectedUrl }" rel="noopener noreferrer" target="_blank">` +
`<span class="ck-media__placeholder__url__text">${ expectedUrl }</span>` +
'<span class="ck ck-tooltip ck-tooltip_s">' +
'<span class="ck ck-tooltip__text">Open media in new tab</span>' +
Expand Down
4 changes: 4 additions & 0 deletions theme/mediaembed.css
Expand Up @@ -19,3 +19,7 @@
from being "squashed" in tight spaces, e.g. in table cells (#44) */
min-width: 15em;
}

.ck-editor__editable:not( .ck-read-only ) .ck-widget:not( .ck-widget_selected ) .ck-media__placeholder {
pointer-events: none;
}

0 comments on commit f012769

Please sign in to comment.