Wcwidth and auto-extend selections

lierdakil commented 9 years ago

This PR includes two major updates:

Using wcwidth to detect wide characters. Current behavior is to treat all non-ascii characters as fullwidth, which breaks with, e.g. Russian or other half-width non-ASCII characters (see 8f0a05e for test cases and 4e258ce for actual implementation)
Extend selections to whole markdown paragraphs (e960df1). Empty selections are extended only if autoSelectEntireDocument is false (1e1f7a0). This allows to format individual tables without properly selecting them. This could have something to do with #14, but that's a wild guess.

Otherwise, it's general code cleanup and format.

Requesting comments.

fcrespo82 commented 9 years ago

With this code, when I try to run the specs it shows me the error:

Error: Cannot find module 'wcwidth'

I don't have node nor npm installed. Shouldn't atom download the requirements automatically?

Do you know what could be happening?

lierdakil commented 9 years ago

Run apm install in package directory. This will install required modules. When installing from Atom package repository, this is done automatically.

fcrespo82 commented 9 years ago

Yep, that done it. Thanks!

Published!

fcrespo82 / atom-markdown-table-formatter