Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Implement @whitespace setter. #42

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Implement @whitespace setter. #42

wants to merge 3 commits into from

Commits on Mar 16, 2019

  1. Implement @whitespace setter. 

    As suggested in #29.  A keyword `@whitespace` can be used to switch
whitespace handling between the tree modes `as-is`, `compact`, and
`removed`.

- [x] Implment the functionality.  Set default mode to `compact` and
  update tests.
- [ ] Decide on the exact syntax; should some keyword change?  Should
  `@ws` be allowed as an alias for `@whitespace`?  Should the
  `removed` alternative be called `trim`?  Or some other word change?
- [ ] Add documentation and more tests / examples.
- [ ] Anything else?
    @kaj
    kaj committed Mar 16, 2019
    Configuration menu
    Copy the full SHA
    dcc4560 View commit details
    Browse the repository at this point in the history

  2. Tabs are spaces. 

    Only spaces (ascii 32), tabs, newlines and carriage returns are
considered whitespace.  Other variants (such as no-break space and
spaces of specific width) will probably only appear in templates if
they are actually wanted in output.
    @kaj
    kaj committed Mar 16, 2019
    Configuration menu
    Copy the full SHA
    24913fa View commit details
    Browse the repository at this point in the history

  3. Allow @ws as an alias for @whitespace.

    @kaj
    kaj committed Mar 16, 2019
    Configuration menu
    Copy the full SHA
    ebe1b34 View commit details
    Browse the repository at this point in the history