Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#40429 closed defect (bug) (worksforme)

Right Click for Paste not allowed - use CTRL+V

Reported by: sdpate48's profile sdpate48 Owned by:
Milestone: 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 7 years ago.
Screen shot
40429.patch (473 bytes) - added by sagarprajapati 7 years ago.

Download all attachments as: .zip

Change History (19)

@sdpate48
7 years ago

Screen shot

#1 @afercia
7 years 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.


7 years ago

#4 @rianrietveld
7 years ago

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

Last edited 7 years ago by rianrietveld (previous) (diff)

#5 @sdpate48
7 years ago

Started occurring with Windows 10 Pro Build 15063

Now at Windows 10 Version 1703
Microsoft Edge 40.15063.0.0

Thanks.

#6 @joedolson
7 years 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.


7 years ago

#8 @sdpate48
7 years 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
7 years 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.


7 years ago

#11 @afercia
7 years ago

  • Milestone changed from Awaiting Review to Future Release

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


7 years ago

#13 @azaozz
7 years 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
7 years 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.

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


7 years ago

#16 @afercia
7 years ago

@azaozz, any news from TinyMCE you're aware of about this issue? Thanks!

#17 @azaozz
7 years ago

  • Milestone Future Release deleted
  • Resolution set to worksforme
  • Status changed from new to closed

@afercia this seems to work properly now. Pasting in Edge with the context menu (right-click) item fall back to text mode, pasting with Ctrl+V does "rich-text" paste.

Closing as worksforme. Seems it's working as well as it can :)

Note: See TracTickets for help on using tickets.