[llvm-commits] [LNT] r165446 - /lnt/trunk/lnt/server/reporting/dailyreport.py

[Daniel Dunbar]

Author: ddunbar
Date: Mon Oct  8 18:02:48 2012
New Revision: 165446

URL: http://llvm.org/viewvc/llvm-project?rev=165446&view=rev
Log:
lnt.server.ui/dailyreport: Change to use static CSS styles.

Modified:
    lnt/trunk/lnt/server/reporting/dailyreport.py

Modified: lnt/trunk/lnt/server/reporting/dailyreport.py
URL: http://llvm.org/viewvc/llvm-project/lnt/trunk/lnt/server/reporting/dailyreport.py?rev=165446&r1=165445&r2=165446&view=diff
==============================================================================
--- lnt/trunk/lnt/server/reporting/dailyreport.py (original)
+++ lnt/trunk/lnt/server/reporting/dailyreport.py Mon Oct  8 18:02:48 2012
@@ -126,7 +126,7 @@
         for day_index,day_runs in enumerate(prior_runs):
             for run in day_runs:
                 machine_runs[(run.machine_id, day_index)] = run
-                
+
         # Build the result table of tests with interesting results.
         self.result_table = []
         for field in self.fields:
@@ -145,7 +145,7 @@
                     # If the result is not "interesting", ignore this machine.
                     if not cr.is_result_interesting():
                         continue
-                    
+
                     # Otherwise, compute the results for all the days.
                     day_results = [cr]
                     for i in range(1, self.num_prior_days_to_include):
@@ -164,9 +164,27 @@
                     field_results.append((test, visible_results))
             self.result_table.append((field, field_results))
 
-    def render(self):
+    def render(self, only_html_body=True):
         env = lnt.server.ui.app.create_jinja_environment()
         template = env.get_template('reporting/daily_report.html')
 
+        # Compute static CSS styles for elements. We use the style directly on
+        # elements instead of via a stylesheet to support major email clients
+        # (like Gmail) which can't deal with embedded style sheets.
+        #
+        # These are derived from the static style.css file we use elsewhere.
+        styles = {
+            "body" : ("color:#000000; background-color:#ffffff; "
+                      "font-family: Helvetica, sans-serif; font-size:9pt"),
+            "table" : ("font-size:9pt; border-spacing: 0px; "
+                       "border: 1px solid black"),
+            "th" : (
+                "background-color:#eee; color:#666666; font-weight: bold; "
+                "cursor: default; text-align:center; font-weight: bold; "
+                "font-family: Verdana; padding:5px; padding-left:8px"),
+            "td" : "padding:5px; padding-left:8px",
+        }
+
         return template.render(
-            report=self, analysis=lnt.server.reporting.analysis)
+            report=self, styles=styles, analysis=lnt.server.reporting.analysis,
+            only_html_body=only_html_body)