Pretty formatter options

[muhqu]

Adds some new formatting options to the Gherkin Formatter, that are currently pending in a separat PR (cucumber/gherkin#331).

• right-align numeric values in tables (addresses Improve formatting for columns containing only numbers: right aligned #85, also in PR Rightalign numeric values #105 )
• center-align step keywords
• preserve a single blank line between steps
• preference page for these formatter options

How it looks like...

nice to have future improvements

• allow for project specific preferences via property page

[muhqu]

💡 any help to get these formatter options configurable for the user on a per-project basis would be much appreciated...

[muhqu]

@ilanpillemer what do you think? …worth pulling?

[ilanpillemer]

On Friday, March 13, 2015, Mathias Leppich notifications@github.com wrote:

@ilanpillemer https://github.com/ilanpillemer what do you think? …worth
pulling? [image: ]

In two minds. I don't like separating pretty format logic from gherkin. But
if gherkin is still doing the syntax validation then perhaps it's not so
bad.

—
Reply to this email directly or view it on GitHub
#107 (comment)
.

[muhqu]

[...] separating pretty format logic from gherkin.

@ilanpillemer I don't know what you mean by that.

If you're referring to the copy of PrettyFormatter.java in this PR. It is the same that is waiting to get merged into gherkin. See cucumber/gherkin#331.

[ilanpillemer]

On Friday, March 13, 2015, Mathias Leppich notifications@github.com wrote:

[...] separating pretty format logic from gherkin.

@ilanpillemer https://github.com/ilanpillemer I don't know what you
mean by that.

If you're referring to the copy of PrettyFormatter.java in this PR. It is
the same that is waiting to get merged into gherkin
https://github.com/cucumber/gherkin/. See cucumber/gherkin#331
cucumber/gherkin#331.

Ok. I see Aslak says they are all on Gherkin3.

I will merge your changes in over the weekend for this one then seeing this
version of Gherkin is clearly going out anyway.

—
Reply to this email directly or view it on GitHub
#107 (comment)
.

[muhqu]

@ilanpillemer thanks for merge!