yogthos/Selmer

Add whitespace control?

Open

#115 opened on Feb 1, 2016

View on GitHub
 (21 comments) (13 reactions) (0 assignees)Clojure (121 forks)batch import
enhancementhelp wanted

Repository metrics

Stars
 (1,042 stars)
PR merge metrics
 (No merged PRs in 30d)

Description

Jinja has a useful feature where you can strip out whitespace that you've added to make your template more readable:

{% for item in seq -%}
    {{ item }}
{%- endfor %}

This will yield all elements without whitespace between them. If seq was a list of numbers from 1 to 9, the output would be 123456789.

http://jinja.pocoo.org/docs/dev/templates/#whitespace-control

In other words, adding a - to get -%} will indicate that all whitespace should be removed between the for/if/other statement and the next character.

Feature request: any interest in adding this to Selmer?

Contributor guide

Research direction
Study the Selmer parser code to understand how it processes tags and whitespace, then implement a ' ' modifier to strip whitespace around tags similar to Jinja.
Tech stack
None
Domain
backend
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
ClojureSelmertemplate parsing
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
65