WordPress.org

Make WordPress Core

Opened 4 months ago

Last modified 4 months ago

#40429 new defect (bug)

Right Click for Paste not allowed - use CTRL+V

Reported by: sdpate48 Owned by:
Milestone: Future Release Priority: high
Severity: normal Version:
Component: TinyMCE Keywords:
Focuses: accessibility Cc:

Description

The visual editor is forcing the user to use CTRL+V/CMD+V for shortcuts to paste contents.

While this is very Mac like it's very unfriendly to people with disabilities who have RSS, carpal tunnel, weak hands wrists arms or other issues moving their hands easily from a mouse to the keyboard and back again.

It also does not comply with accessibility guidelines https://msdn.microsoft.com/en-us/library/ms971323.aspx

Attachments (2)

Ctrl V.JPG (132.6 KB) - added by sdpate48 4 months ago.
Screen shot
40429.patch (473 bytes) - added by sagarprajapati 4 months ago.

Download all attachments as: .zip

Change History (16)

@sdpate48
4 months ago

Screen shot

#1 @afercia
4 months ago

  • Component changed from General to TinyMCE
  • Version 4.7.3 deleted

Seems related to some TinyMCE feature.

This ticket was mentioned in Slack in #accessibility by rianrietveld. View the logs.


4 months ago

#4 @rianrietveld
4 months ago

@sdpate48 can you please add which operating system and browser you are using?
And also the versions?

Last edited 4 months ago by rianrietveld (previous) (diff)

#5 @sdpate48
4 months ago

Started occurring with Windows 10 Pro Build 15063

Now at Windows 10 Version 1703
Microsoft Edge 40.15063.0.0

Thanks.

#6 @joedolson
4 months ago

  • Priority changed from normal to high

I've reproduced this scenario in Edge/Windows 10. Doesn't happen in other browsers in Windows 10; so this appears to be an Edge specific issue.

Changing status on this to high; I think this is a major issue, but we'll have to investigate what Edge is doing differently here.

This ticket was mentioned in Slack in #core-editor by afercia. View the logs.


4 months ago

#8 @sdpate48
4 months ago

Sorry to add another issue - Ctrl V is creating screen garbage in Visual Mode

http://njnnetwork.com/njn/wp-content/uploads/2017/04/Word-Press-Copy-Paste.jpg

#9 @sagarprajapati
4 months ago

  • Keywords has-patch added

Hi All,

I have checked it in detail and found that clipboardContent["text/html"] = getPasteBinHtml(); code is not getting proper content and here we also no need to add the data for ie becuase it's already get the data in insertClipboardContent() funcation. I have attached patch for your review please check it.

Thanks

This ticket was mentioned in Slack in #accessibility by afercia. View the logs.


4 months ago

#11 @afercia
4 months ago

  • Milestone changed from Awaiting Review to Future Release

This ticket was mentioned in Slack in #core-tinymce by afercia. View the logs.


4 months ago

#13 @azaozz
4 months ago

  • Keywords has-patch removed

@sdpate48 thanks for the bug report.

The visual editor is forcing the user to use CTRL+V/CMD+V to paste contents.

This is not the case. The visual editor shows the error message when pasting by using the context (right-click) menu fails. This is not new. That message has been there for a while. Seems that a recent security related change in Edge has started to trigger it.

I agree that the text of that error message can be better. Seems it will be updated in the next TinyMCE version in 4.8. Also the browser menu can still be used, only the context menu fails. See https://wordpress.slack.com/archives/C0UCMQP0F/p1493685974261191.

@sagarprajapati not sure if this patch would work. It prevents pasting of HTML when the clipboard cannot be accessed. Then pasting from the context menu will be "plain text" but pasting with Ctrl+V or the browser menu will be with HTML tags and look better. Then the problems will be: "When I paste from the context menu, all tags disappear!" :)

In any case this has to be fixed or improved upstream. Better to make a PR for TinyMCE: https://github.com/tinymce/tinymce. Maybe pasting the plain text and then showing an error message explaining why it is just plain text would be better user experience?

#14 @afercia
4 months ago

Pasting with the right click contextual menu seems to work for me on some of the TinyMCE demos (the ones that don't enable the TinyMCE custom contextual menu). For example: https://www.tinymce.com/docs/demo/valid-elements/
Tested on Win 10, Edge.

Note: See TracTickets for help on using tickets.