Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

'Trailing white space' plugin should ignore files with .MD extension #62

Closed
vinyar opened this issue Mar 20, 2015 · 5 comments
Closed

Comments

@vinyar
Copy link

vinyar commented Mar 20, 2015

Trailing white spaces are part of the Markdown syntax.

As mentioned many times in this bug: #10
Markdown files should be filtered out of the plugin 'magic'

@vinyar
Copy link
Author

vinyar commented Mar 20, 2015

A very interesting positive side effect - after disabling the plugin, the markdown preview generation is substantially faster

@notslang
Copy link

just add this to your config:

".gfm.source":
  whitespace:
    removeTrailingWhitespace: false

The docs for scoped settings are here: http://blog.atom.io/2014/10/31/language-scoped-config.html

From what I've seen, most users don't write markdown that makes use of trailing white-spaces to indicate formatting, so it doesn't make sense to have this be off by default.

@ccoenen
Copy link
Contributor

ccoenen commented Mar 20, 2015

@slang800 dismissing a feature of a grammar (br-tags with two spaces at the EOL) based on your perspective is hardly useful.

With that attitude, i could say "hey, i don't know anyone who uses closures in JavaScript. Let's just strip them". I'm exxagerating, of course, but it IS a feature of MarkDown. And you don't get to choose if it is important or not.

@vinyar
Copy link
Author

vinyar commented Mar 22, 2015

@slang800 Thanks. This is good info for power users. I used Atom for a few days now, and it seems like a pretty major oversight to support a language but ignore some of its base syntax (thus this bug/enhancement) request.
@ccoenen +1

@lee-dohm
Copy link
Contributor

lee-dohm commented Jul 9, 2015

To my understanding this is currently baked into the code here:

https://github.com/atom/whitespace/blob/master/lib/whitespace.coffee#L59-L63

Because of this, I'm going to close this Issue. If I'm making a mistake, I trust everyone to tell me 😀

@lee-dohm lee-dohm closed this as completed Jul 9, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants