search

Zettt / km-markdown-library

Markdown library for Keyboard Maestro.
193 stars 10 forks source link

formd with MacVim #7

Closed fncll closed 10 years ago

fncll commented 10 years ago

When I use the formd macro in MacVim, I end up with duplicated text. It works fine in TextEdit.

For instance, if I start with this:

This is a [test](http://foo.com) plus some more.

And run the formd macro, I end up with:

This is a [test][1] plus some more.

[1]: http://foo.comThis is a [test](http://foo.com) plus some more.

The formd variable has the correct beginning text input (I checked using a debug breakpoint).

Zettt commented 10 years ago

Can't replicate this problem. Works fine here.

fncll commented 10 years ago

Again, to clarify, you can't replicate with MacVim specifically?

Zettt commented 10 years ago

No.

fncll commented 10 years ago

That's again an ambiguous answer as it could mean: no, I can't replicate with MacVim specifically or no I didn't try try with MacVim specifically.

Based on your curt replies, I'm not sure if I have offended you in some way or what, but I am sorry for asking for clarifications and I will refrain from reporting any problems or asking any questions in the future.

I'm not sure why you have the issues reporting feature turned on in gitHub if you are going to be singularly unhelpful when someone uses it.

Zettt commented 10 years ago

I don't have this problem with MacVim. Sorry if I was a bit brief, but I'm just a tad busy with work at the moment.

fncll commented 10 years ago

My apologies for misunderstanding.

I found the problem with this macro: your macro first sets the current application to the variable:

MMD__Current App

Then it checks whether that App is MacVim using a different variable (typo):

MMD_Current App

Also, I changed the logic to "if all of the following are true" since they are each "does not match" conditions...

Not sure how it is working for you, since that typo makes the macro skip the code that checks for MacVim! By fixing that and the accompanying logic, it works fine now.

Zettt commented 10 years ago

Glad you got it working. With regards to your fix. Are you certain this is the formd macro we are talking about here? I'm asking because I plan to release the next version today and want to get this fix into the release.

fncll commented 10 years ago

My apologies: it's the footnote macro (#32)

Zettt commented 10 years ago

Gotcha. This is fixed since version 2.1.1. The most current version is 2.1.2. I guess that means I can close this issue now. Thanks for the help!