Skip to content
This repository was archived by the owner on Sep 11, 2024. It is now read-only.

Add support for pasting files into the text box#605

Merged
ara4n merged 1 commit into
matrix-org:developfrom
vijfhoek:feature/pastefiles
Mar 9, 2017
Merged

Add support for pasting files into the text box#605
ara4n merged 1 commit into
matrix-org:developfrom
vijfhoek:feature/pastefiles

Conversation

@vijfhoek
Copy link
Copy Markdown
Contributor

@vijfhoek vijfhoek commented Jan 8, 2017

Only supports the new rich-text-supporting text editor.

Works on Firefox (51) and Chrome (55). Doesn't work on Safari (10.0.2), although that seems to be a dash.js issue. Needs further investigation.

Fixes element-hq/element-web#1297.

Only supports the new rich-text-supporting text editor
@matrixbot
Copy link
Copy Markdown
Member

Can one of the admins verify this patch?

@vijfhoek vijfhoek changed the title Add support for pasting into the text box Add support for pasting files into the text box Jan 8, 2017
@thepaka
Copy link
Copy Markdown

thepaka commented Feb 27, 2017

Hi,
Can someone takes a look on this feature ?
It works (for me) and the code changes seems to be safe to merge.

@ara4n
Copy link
Copy Markdown
Member

ara4n commented Mar 9, 2017

sorry for the delay in reviewing; was worried that it was RTE-only, but given that RTE is about to land, now is a perfect time to land this too. thanks!

LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants