Opened 20 months ago

Closed 18 months ago

Last modified 18 months ago

#16857 closed Bug (fixed)

Ctrl + Shift + V blocked by copy formatting

Reported by: Wiktor Walc Owned by: kkrzton
Priority: Nice to have (we want to work on it) Milestone: CKEditor 4.7.0
Component: General Version: 4.6.0
Keywords: Cc:

Description (last modified by Marek Lewandowski)

Steps to reproduce

  1. Open "Document Editor" sample on ckeditor.com.
  2. Copy something, e.g. a part of a header.
  3. Press Ctrl + Shift + V to paste clipboard as plain text.

Expected result

Text is pasted.

Actual result

Text is not pasted.

Other details (browser, OS, CKEditor version, installed plugins)

We'd like implement this in a way that CTRL+SHIFT+V will paste formatting only if it has been activated with CTRL+SHIFT+C - so that the Copy Formatting icon is active, and the toolbar button shows "pressed" state.

If copy formatting is not active, then this hotkey will paste as plain text.

This will improve interoperability with apps like MS Word, which use exact same hotkey, while keeping the original paste as plain text hotkey for people that doesn't use copy formatting.

Issue caused by copy formatting using keyboard shortcut which is usually used to paste plain text browsers.

Change History (7)

comment:1 Changed 20 months ago by Marek Lewandowski

Milestone: CKEditor 4.7.0
Priority: NormalNice to have (we want to work on it)
Status: newconfirmed

comment:2 Changed 20 months ago by Tade0

Owner: set to Tade0
Status: confirmedreview

Solution was to return false after putting screen reader message that applying failed.

This way the keystroke handler knows not to prevent the default behaviour which is pasting plain text.

Changes pushed to branch:t/16857.

comment:3 Changed 19 months ago by Marek Lewandowski

Resolution: fixed
Status: reviewclosed

Fixed with git:caa3bcb3c2.

comment:4 Changed 19 months ago by Marek Lewandowski

Description: modified (diff)

comment:5 Changed 18 months ago by Marek Lewandowski

Resolution: fixed
Status: closedreopened

This fix no longer works after #16954, which surfaced underlying issue, that multiple command can not be hooked assigned to the same keystroke.

comment:6 Changed 18 months ago by kkrzton

Owner: changed from Tade0 to kkrzton
Status: reopenedreview

comment:7 Changed 18 months ago by Marek Lewandowski

Resolution: fixed
Status: reviewclosed

Rebased and merged (ff) using github. Diff available at https://github.com/ckeditor/ckeditor-dev/pull/341/files.

Last edited 18 months ago by Marek Lewandowski (previous) (diff)
Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy