#28088 closed defect (bug) (fixed)
wpviews: Safari: unable to copy/paste
Reported by: | iseulde | Owned by: | azaozz |
---|---|---|---|
Milestone: | 4.0 | Priority: | normal |
Severity: | normal | Version: | 3.9 |
Component: | TinyMCE | Keywords: | has-patch |
Focuses: | Cc: |
Description
- In Safari, you can't copy/paste a wpview after selecting it.
- If you put your cursor in a paragraph before or after a wpview, the wpview will be selected visually, but the cursor moves to the start of the first paragraph.
- Copying everything in the editor does not include the wpviews.
It looks like TinyMCE fails to select a contenteditable="false"
area in Safari on line 96 with editor.selection.select( clipboard, true )
.
Attachments (1)
Change History (11)
#1
@
10 years ago
- Summary changed from Safari: wpviews: unable to copy/paste to wpviews: Safari: unable to copy/paste
#5
@
10 years ago
- Owner set to azaozz
- Resolution set to fixed
- Status changed from new to closed
In 28565:
#6
follow-up:
↓ 7
@
10 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
This will need to be .wpview-wrap .wpview-clipboard *
if you want html elements inside the clipboard to be selectable (not just text).
#7
in reply to:
↑ 6
@
10 years ago
Replying to avryl:
Hmm, there shouldn't be any html elements inside the clipboard? The original string (shortcode) together with any html is ran through encodeURIComponent()
which encodes pretty much everything except [^a-zA-Z0-9_-]
.
#8
@
10 years ago
Why not? The clipboard element actually contains the decoded text. :)
https://core.trac.wordpress.org/browser/trunk/src/wp-includes/js/tinymce/plugins/wpview/plugin.js#L40
This is caused by:
28088-01.patch fixes the issue.