Opened 12 years ago
Closed 12 years ago
#554 closed enhancement (fixed)
Meddleware improvments & consider merging my trim_whitespace meddleware plugin
Reported by: | spaetz | Owned by: | spaetz |
---|---|---|---|
Priority: | major | Milestone: | 0.3.2 |
Component: | programming | Keywords: | |
Cc: | Parent Tickets: |
Description
1) Our Meddleware contains a NoOp Meddleware. This seems rather superfluous
2) Our ENABLED_MEDDLEWARE is stored in a tuple rather than a list, so it is not
modifiable.
3) trim_whitespace meddleware plugin
Our HTML output is very verbose (=whitespacy) as our templates are
written with an 80 char limit and lots of newlines between blocks,
variables etc....
This is a plugin that naively strips of all but the first whitespace
from the HTML response. We might want to have an all-fancy html tidy
interface here at some point, but it nicely decreases the HTML size
about a third on some simple pages.
The formerge/trim_whitespace_plugin branch on git://gitorious.org/~spaetz/mediagoblin/spaetz-mediagoblin.git addresses these 3 issues in 3 patches.
It would be nice to merge at least no 2 (make ENABLED_MEDDLEWARES modifiable) for this release, so my plugin works and can be distributed separatedly).
What is our policy with regard to plugins? Ship some bundled? Or should I provide them separately?
Change History (2)
comment:1 by , 12 years ago
Milestone: | 0.3.3 → 0.3.2 |
---|---|
Owner: | set to |
Status: | new → assigned |
comment:2 by , 12 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Okay, I think, this one was merged too. See 99454686 and its ancestors.
Can someone please confirm?