From 1067c208cb852de1f9feb924685df32596e2a758 Mon Sep 17 00:00:00 2001
From: Samuel Cochran <sj26@sj26.com>
Date: Fri, 23 Dec 2011 03:34:02 +0800
Subject: [PATCH] Pretty up XSL, cater to empty case

---
 public/stylesheets/analysis.xsl | 51 +++++++++++++++++++--------------
 1 file changed, 29 insertions(+), 22 deletions(-)

diff --git a/public/stylesheets/analysis.xsl b/public/stylesheets/analysis.xsl
index 81713f6..00ad394 100644
--- a/public/stylesheets/analysis.xsl
+++ b/public/stylesheets/analysis.xsl
@@ -1,26 +1,33 @@
 <?xml version="1.0" encoding="ISO-8859-1"?>
 <xsl:stylesheet version="1.0" xmlns:xsl="http://www.w3.org/1999/XSL/Transform">
   <xsl:template match="/xml">
-  
-  <section class="fractal-analysis">
-  <xsl:for-each select="result/errors/error">
-    <xsl:variable name="errorCount" select="position()"/>
-    <p class="report-intro">
-      <strong><xsl:value-of select="$errorCount" />.</strong> The <span class="code"><xsl:value-of select="rule/rule_name" /></span> near <span class="code"><xsl:value-of select="snippet" /></span> on line <span class="code"><xsl:value-of select="error_line" /></span> of your code.
-    </p>
-    <dl class="unsupported-clients">
-      <dt>This is unsupported in:</dt>
-      <dd>
-        <ul>
-          <xsl:for-each select="rule/email_client//item">
-          <li>
-            <xsl:value-of select="." />
-          </li>
-          </xsl:for-each>
-        </ul>    
-      </dd>
-    </dl>
-  </xsl:for-each>
-  </section>
+    <section class="fractal-analysis">
+      <h1>Fractal Results</h1>
+      <xsl:choose>
+        <xsl:when test="not(result/errors/node())">
+          <p class="report-intro valid">Your HTML and CSS for your email is valid.</p>
+        </xsl:when>
+        <xsl:otherwise>
+          <p class="report-intro invalid">Your HTML and CSS for your email contains potential errors:</p>
+          <ol>
+            <xsl:for-each select="result/errors/error">
+              <li>
+                <p class="error-intro">The <code><xsl:value-of select="rule/rule_name" /></code> near <code><xsl:value-of select="snippet" /></code> on line <code><xsl:value-of select="error_line" /></code> of your code.</p>
+                <dl class="unsupported-clients">
+                  <dt>This is unsupported in:</dt>
+                  <dd>
+                    <ul>
+                      <xsl:for-each select="rule/email_client//item">
+                        <li><xsl:value-of select="." /></li>
+                      </xsl:for-each>
+                    </ul>
+                  </dd>
+                </dl>
+              </li>
+            </xsl:for-each>
+          </ol>
+        </xsl:otherwise>
+      </xsl:choose>
+    </section>
   </xsl:template>
-</xsl:stylesheet>
\ No newline at end of file
+</xsl:stylesheet>