GNOME Bugzilla – Bug 702302
indent outputted html
Last modified: 2014-02-01 21:23:12 UTC
The attached patch against commit a3c05d8c8c1c30ee5c0430c0c331fda434845ecd Author: David Schleef <ds@schleef.org> Date: Thu May 2 12:35:44 2013 -0700 server: convert to jpeg snapshot replacements Modifies the GSS_A and GSS_P macros (and adds a few others) to keep track of what indentation level the generated HTML is at, and properly indent subsequently generated HTML. This is primarily to make it easy to read the page source in the browser.
Created attachment 246864 [details] [review] Patch to add indenting logic to html output.
Created attachment 247664 [details] [review] Updated patch This version of the patch corrects a few indentation problems.
Not going to apply this. The GSS_A and _P macros are a large enough barrier to getting HTML into the code. It's quite reasonable to expect a user to reformat the HTML if they want to read it. That being said, I'd be cool with an HTML formatter built-in to GSS that will optionally reformat after the file is generated.
Michael, is that something you would be interested in implementing as well?
@Tim-Philip I'm afraid not. My original patch was developed as part of a project for my employer, and split out from that work to submit to GSS in the hope that others would find it useful. We've since stopped the project that resulted in that patch, and at the moment have no plans to revisit it. If the situation changes, we will consider this request. --Mike