Investigate if Beautifier impacts page load-time, if results are cached and/or cacheable and if no, implement cache strategy for beautified pages.

CommentFileSizeAuthor
#7 beautify-484756.patch3.78 KBmikeytown2
beautify.module.patch343 bytesmikeytown2

Comments

mikeytown2’s picture

mikeytown2 commented
Status: Active » Needs review
psynaptic’s picture

psynaptic commented

Looks good thanks. Would you like CVS access so you can commit this yourself?

mikeytown2’s picture

mikeytown2 commented

sure, but I might end up fixing other things as well. Just a FYI

psynaptic’s picture

psynaptic commented

That is the kind of warning I love! I have granted you access, feel free to do whatever you think is right.

I think we should probably remove the search and replace processing provided by this module (the bits where it removes newlines) as it messes up content submitted via textareas inputs.

mikeytown2’s picture

mikeytown2 commented
Status: Needs review » Reviewed & tested by the community
mikeytown2’s picture

mikeytown2 commented
Status: Reviewed & tested by the community » Needs work
mikeytown2’s picture

mikeytown2 commented
Status: Needs work » Needs review
StatusFileSize
new beautify-484756.patch3.78 KB

works with pressflow now

matthieuscarset’s picture

matthieuscarset commented
Title: Page Cache » Cache strategy
Version: 6.x-1.x-dev » 1.0.1
Assigned: Unassigned » matthieuscarset
Category: Bug report » Feature request
Issue summary: View changes
Status: Needs review » Active

This is still a relevant feature request with our new version 1.0.0 for D8/9/10.

I reopen the issue and I updated the title and description to reflect our current state/code.

matthieuscarset’s picture

matthieuscarset commented