[Mpuls-commits] r3916 - in base/trunk: . mpulsweb/templates/phase

[scm-commit@wald.intevation.org]

Author: torsten
Date: 2010-10-07 10:53:32 +0200 (Thu, 07 Oct 2010)
New Revision: 3916

Modified:
   base/trunk/ChangeLog
   base/trunk/mpulsweb/templates/phase/phase.mako
Log:
Issue939: Use icons instead of colors to indicate missing fields or errors


Modified: base/trunk/ChangeLog
===================================================================
--- base/trunk/ChangeLog	2010-10-07 08:44:01 UTC (rev 3915)
+++ base/trunk/ChangeLog	2010-10-07 08:53:32 UTC (rev 3916)
@@ -9,6 +9,8 @@
 
 	* mpulsweb/templates/casemanagement/search.mako: Issue1260: Fixed
 	listing of phases in search.
+	* mpulsweb/templates/phase/phase.mako: Issue939: Use icons instead of
+	colors on phase page to indicate missing fields or other errors.
 
 2010-10-06  Bernhard Herzog  <bh at intevation.de>
 

Modified: base/trunk/mpulsweb/templates/phase/phase.mako
===================================================================
--- base/trunk/mpulsweb/templates/phase/phase.mako	2010-10-07 08:44:01 UTC (rev 3915)
+++ base/trunk/mpulsweb/templates/phase/phase.mako	2010-10-07 08:53:32 UTC (rev 3916)
@@ -19,7 +19,7 @@
       <% is_allowed = h.hasRole(('cm_ka',)) %>
       <tr>
         <th>${_('cm_info_phase')}</th>
-        <th width="15px">&nbsp;</th>
+        <th width="25px">&nbsp;</th>
         <th width="50%">${_('case_cm_phase_required_fields')}</th>
         <th>${_('case_cm_phase_time')}</th>
         % if c.show_actions:
@@ -58,12 +58,13 @@
             % endif
             </span>
           </td>
+          <td class="middle">
           % if p1.is_ok() and (c.phases.is_on_path(p1.id, current_phase,True) or c.phases.is_on_path(current_phase, p1.id, True)): 
-
-          <td class="ok"></td>
+            <img src="/images/icons/info/ok_22.png" alt="Ok">
           % else:
-          <td class="error"></td>
+            <img src="/images/icons/info/failure_22.png" alt="Error">
           % endif
+          </td>
           <td class="note" style="border-bottom: 1pt dotted; border-color: #000;">
             <% pages, order = p1.getLinks() %>
             % if len(pages) > 0:
@@ -93,11 +94,13 @@
           % endif
         </tr>
         <tr class="${(num%2) and 'hl' or ''}">
+          <td class="middle">
           % if p2.is_ok() and (c.phases.is_on_path(p2.id, current_phase, True) or c.phases.is_on_path(current_phase, p2.id, True)): 
-          <td class="ok"></td>
+            <img src="/images/icons/info/ok_22.png" alt="Ok">
           % else:
-          <td class="error"></td>
+            <img src="/images/icons/info/failure_22.png" alt="Error">
           % endif
+          </td>
           <td class="note">
             <% pages, order = p2.getLinks() %>
             % if len(pages) > 0: