Skip to content
This repository was archived by the owner on May 12, 2026. It is now read-only.

replace selection with pasted image#1119

Merged
kohei-takata merged 2 commits into
BoostIO:masterfrom
mslourens:paste-image
Nov 15, 2017
Merged

replace selection with pasted image#1119
kohei-takata merged 2 commits into
BoostIO:masterfrom
mslourens:paste-image

Conversation

@mslourens

Copy link
Copy Markdown
Contributor

fixes issue #1086 - only paste the created image tag over the current selection, not the start and end of the whole textarea.

@mslourens

Copy link
Copy Markdown
Contributor Author

Can someone with knowledge about the tests have a look at my failing build, because I don't know why it's failing, it succeeded on my pc. This test is failing reorderFolder-test with this error message: TypeError: Cannot read property 'key' of undefined

@kazup01

kazup01 commented Nov 14, 2017

Copy link
Copy Markdown
Member

@mslourens Thank you for your contribution!
After "restart build", I could fix the failing.
https://travis-ci.org/BoostIO/Boostnote/builds/301640651

We will review this pull request!

@kazup01 kazup01 added the awaiting review ❇️ Pull request is awaiting a review. label Nov 14, 2017
@kazup01 kazup01 requested a review from kohei-takata November 14, 2017 04:09

@kohei-takata kohei-takata left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mslourens Thanks for your PR, and please check the comment!

Comment thread browser/components/CodeEditor.js Outdated
@@ -214,8 +214,7 @@ export default class CodeEditor extends React.Component {

insertImageMd (imageMd) {
const textarea = this.editor.getInputField()

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you delete this line?
textarea is not used I think.

@kazup01 kazup01 added awaiting changes 🖊️ Pull request has been reviewed, but contributor needs to make changes. and removed awaiting review ❇️ Pull request is awaiting a review. labels Nov 14, 2017
@mslourens

Copy link
Copy Markdown
Contributor Author

@kohei-takata I fixed my code according to your review comment, you can review it again.

@kazup01 kazup01 added awaiting review ❇️ Pull request is awaiting a review. and removed awaiting changes 🖊️ Pull request has been reviewed, but contributor needs to make changes. labels Nov 15, 2017
@kazup01

kazup01 commented Nov 15, 2017

Copy link
Copy Markdown
Member

Thank you @mslourens :)

@kohei-takata kohei-takata left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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

Labels

None yet

4 participants