New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature UnArrange or Compact command #7

Closed
fanglingsu opened this Issue Sep 23, 2011 · 2 comments

Comments

Projects
None yet
2 participants
@fanglingsu
Contributor

fanglingsu commented Sep 23, 2011

I would find it nice to have to possibility to compact a CSV File after I Arraged it, so that the additional spaces will be removed around the delimiters?

Would this be possible to do?

@chrisbra

This comment has been minimized.

Show comment
Hide comment
@chrisbra

chrisbra Sep 24, 2011

Owner

Thanks for the suggestion. I just pushed a change. Please check current master.
I hesitated to implement such a feature, since it is hard to really undo the :ArrangeColumn command.
But I think it would be good enough so simply delete leading blanks and that is what the :UnArrangeColumn command does.
(This might differ, if you had a column containing only blanks, before the :ArrangeColumn command. After an :UnArrangeColumn, this column will be empty.)

Owner

chrisbra commented Sep 24, 2011

Thanks for the suggestion. I just pushed a change. Please check current master.
I hesitated to implement such a feature, since it is hard to really undo the :ArrangeColumn command.
But I think it would be good enough so simply delete leading blanks and that is what the :UnArrangeColumn command does.
(This might differ, if you had a column containing only blanks, before the :ArrangeColumn command. After an :UnArrangeColumn, this column will be empty.)

@fanglingsu

This comment has been minimized.

Show comment
Hide comment
@fanglingsu

fanglingsu Sep 25, 2011

Contributor

Pretty nice work! It works fine. And for me it doesn't matter if previous whitespace will be stripped away.

Maybe CompactColumn would be the better name, because UnArrangeColumn does not undo ArrangeColumn, but I don't think UnArrangeColumn is wrong.

Contributor

fanglingsu commented Sep 25, 2011

Pretty nice work! It works fine. And for me it doesn't matter if previous whitespace will be stripped away.

Maybe CompactColumn would be the better name, because UnArrangeColumn does not undo ArrangeColumn, but I don't think UnArrangeColumn is wrong.

@fanglingsu fanglingsu closed this Sep 25, 2011

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment