Opened 5 years ago

Closed 5 years ago

#12264 closed New Feature (fixed)

Files in dataTransfer

Reported by: Piotr Jasiun Owned by: Piotr Jasiun
Priority: Normal Milestone: CKEditor 4.5.0 Beta
Component: General Version:
Keywords: Cc:

Description (last modified by Piotr Jasiun)

Part of: #11437.

dataTransfer should contains files.

Change History (9)

comment:1 Changed 5 years ago by Piotr Jasiun

Description: modified (diff)
Status: newconfirmed

comment:2 Changed 5 years ago by Piotr Jasiun

Owner: set to Piotr Jasiun
Status: confirmedassigned

comment:3 Changed 5 years ago by Piotr Jasiun

Description: modified (diff)
Summary: File APIFiles in dataTransfer

comment:4 Changed 5 years ago by Piotr Jasiun

Milestone: CKEditor 4.5.0

comment:5 Changed 5 years ago by Piotr Jasiun

Status: assignedreview

Changes in t/12264.

comment:6 Changed 5 years ago by Piotr Jasiun

There is one case we could handle in this ticket: paste file on Chrome. We can get such file using:

evt.clipboardData.items[0].getAsFile()

But it looks more like a bug:

  • files is empty
  • item type is plain/text instead of file mime type (what makes it hard to recognize),
  • there is always only 1 item (even if I paste multiple files).

So I believe we should skip this case and wait until Chrome will put that file into files array.

comment:7 Changed 5 years ago by Piotr Jasiun

Rebased on the newest #12172.

comment:8 Changed 5 years ago by Piotr Jasiun

Rebased on the newest #12172.

comment:9 Changed 5 years ago by Piotrek Koszuliński

Resolution: fixed
Status: reviewclosed

Basic implementation, so nothing to check for now (except JSLint error ;)). We'll need to update documentation afterwards.

Merged to major with git:87be663.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy