Strange text duplicates when clicking a button several times

[nomad5000]

Reproduce:

select a part of the text that you want to apply style to.

Then click the button (bold,...) then click the button again, again and again.

Result will be several times insert of partial test originaly selected + markdown to apply

[jrm2k6]

Thanks for the catch! Will check.

[nomad5000]

Yes have a fix for this don't know if it is good style. Just drop selection
in the handler for each button

Am Donnerstag, 3. November 2016 schrieb Jeremy Dagorn :

Thanks for the catch! Will check.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#35 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEAaUFe7Twu4sRPcoinwviNPQaA7m5tIks5q6gwSgaJpZM4KoQsO
.

[jrm2k6]

Got it. Will wait for your PR.

[nomad5000]

hey,

how do I run the tests?

I get following error when doing npm test

FAIL __tests__/markdownEditor-test.js ● Runtime Error SyntaxError: Unexpected token ] at createMockFunction (/Users/michael/GitRepos/react-markdown-editor/node_modules/jest-mock/src/index.js:163:10) at makeComponent (/Users/michael/GitRepos/react-markdown-editor/node_modules/jest-mock/src/index.js:246:9) at generateMock (/Users/michael/GitRepos/react-markdown-editor/node_modules/jest-mock/src/index.js:293:16) at getSlots.forEach.slot (/Users/michael/GitRepos/react-markdown-editor/node_modules/jest-mock/src/index.js:303:20) at Array.forEach (native) at generateMock (/Users/michael/GitRepos/react-markdown-editor/node_modules/jest-mock/src/index.js:298:30) at getSlots.forEach.slot (/Users/michael/GitRepos/react-markdown-editor/node_modules/jest-mock/src/index.js:303:20) at Array.forEach (native) at generateMock (/Users/michael/GitRepos/react-markdown-editor/node_modules/jest-mock/src/index.js:298:30) at getSlots.forEach.slot (/Users/michael/GitRepos/react-markdown-editor/node_modules/jest-mock/src/index.js:303:20)

[nomad5000]

OK Solved there was a mock problem with the markup package.

I would like to submit a PR.

Do not have permission to push a branch though

[nomad5000]

Actually I checked github's md editor.

It works as follows:

• Select text
• Apply Button
• Markdown around the selected text
• Text stays selected
• Press Button again
• Markdown around text is removed

[jrm2k6]

Interesting. Thanks for looking into that.
So far I used this package in two projects without issues, but I can see I need to work on adding more features.
If you want to submit a PR you should fork the repo, create a branch on your fork and submit a PR later.

Can I ask in what context you are using this package?

[nomad5000]

Hi,
I want to start using react and was building my first use case.

Then I required a md editor.

So I started to look around.

I found yours was the easiest to install.

So now I'm wondering if I could just do 2 things at once.

Contribute and learn react :)

That's the reason I'm interested

Am Freitag, 4. November 2016 schrieb Jeremy Dagorn :

Interesting. Thanks for looking into that.
So far I used this package in two projects without issues, but I can see I
need to work on adding more features.
If you want to submit a PR you should fork the repo, create a branch on
your fork and submit a PR later.

Can I ask in what context you are using this package?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#35 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEAaUBOgzCyYvOjPBG6-19YESPEjV6Grks5q61rRgaJpZM4KoQsO
.

[jrm2k6]

Alright, do not hesitate to fork the repo, work on it and let me know if you have any questions!

[nomad5000]

Yes I will,

One thing. I want to change the code of the library and use the updated
library when running the examples. Is there a proper way to do that.

Right now I always have to

Rm -rf node_modules

Then rpm i

That sucks terribly

Am Freitag, 4. November 2016 schrieb Jeremy Dagorn :

Alright, do not hesitate to fork the repo, work on it and let me know if
you have any questions!

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#35 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEAaUJx5k-4s2PEp8CHbLgYAq-D14nEFks5q62ESgaJpZM4KoQsO
.

[jrm2k6]

You should be able to use npm link: https://docs.npmjs.com/cli/link

[nomad5000]

I tried but I get some errors that I can't fix with that approach

Am Freitag, 4. November 2016 schrieb Jeremy Dagorn :

You should be able to use npm link: https://docs.npmjs.com/cli/link

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#35 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEAaUHIYoPr1vdnts7oKmRKIZ6t4EiuCks5q63_DgaJpZM4KoQsO
.

[jrm2k6]

What are the errors you are receiving?

On Fri, Nov 4, 2016 at 12:01 PM nomad5000 notifications@github.com wrote:

I tried but I get some errors that I can't fix with that approach

Am Freitag, 4. November 2016 schrieb Jeremy Dagorn :

You should be able to use npm link: https://docs.npmjs.com/cli/link

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<
#35 (comment)
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AEAaUHIYoPr1vdnts7oKmRKIZ6t4EiuCks5q63_DgaJpZM4KoQsO

.

—
You are receiving this because you were assigned.

Reply to this email directly, view it on GitHub
#35 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABCaY98xbUbWCLTlW1rjv5_IKjezncb9ks5q64DwgaJpZM4KoQsO
.