Changeset 5957


Ignore:
Timestamp:
Feb 14, 2012, 11:13:27 AM (10 years ago)
Author:
Nicklas Nordborg
Message:

References #1655: GUI improvements

  • List of annotations and failures/warnings in the 'Item overview' details.
  • Bottom border in 'Inherit annotations' dialog.
Location:
trunk/www/common
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/www/common/annotations/inherit.jsp

    r5919 r5957  
    375375      %>
    376376      <h1><%=title%> <base:help helpid="annotations.inherit" /></h1>
    377       <div class="content">
     377      <div class="content bottomborder">
    378378      <%
    379379    }
  • trunk/www/common/overview/failures.jsp

    r5951 r5957  
    107107  </script>
    108108  <style>
    109   .failure td {
     109  tr.failure
     110  {
     111    border-bottom: 1px dotted #A0A0A0;
     112    border-top: 1px dotted #A0A0A0;
     113    cursor: pointer;
     114  }
     115  tr.failure:first-child
     116  {
     117    border-top: 1px dotted transparent;
     118  }
     119  tr.failure td
     120  {
    110121    vertical-align: bottom;
    111     border-bottom: 1px dotted #999999;
    112122    padding: 2px;
    113123  }
    114   .index {
     124  .index
     125  {
    115126    font-weight: bold;
    116127    text-align: right;
    117128  }
    118   .summary {
    119     border-bottom: 1px solid #999999;
    120     padding-top: 2px;
    121     padding-bottom: 2px;
    122     xposition: fixed;
    123     xtop: 0px;
    124     width:100%;
    125     background: #ffffff;
    126   }
     129
    127130  </style>
    128131  </base:head>
     
    131134    <div class="absolutefull" onmousemove="parent.DragDrop.drag(event)" onmouseup="parent.DragDrop.endDrag(event)" >
    132135   
    133       <div class="summary">
    134       <b>
    135       <%
    136       if (showFailures)
    137       {
     136      <div class="absolutefull filled" style="height: 2em; padding-left: 1em;">
     137      <table style="width: 100%; height: 100%;"><tr><td><b>
     138        <%
     139        if (showFailures)
     140        {
     141          %>
     142          <%=errors.size() %> error(s),
     143          <%=warnings.size()%> warning(s)
     144        <%
     145        }       
     146        else
     147        {
     148          %>
     149          <i>Item-tree not validated</i>
     150          <%
     151        }
    138152        %>
    139         <%=errors.size() %> error(s),
    140         <%=warnings.size()%> warning(s)
    141       <%
    142       }       
    143       else
    144       {
    145         %>
    146         <i>Item-tree not validated</i>
    147         <%
    148       }
    149       %>
    150153      </b>
     154      </td></tr></table>
    151155      </div>
    152156      <%
     
    154158      {
    155159        %>
    156         <br>
    157         <div class="joust">
    158         <table border="0" cellspacing="0" cellpadding="0" width="100%">
     160        <div class="absolutefull topborder" style="top: 2em;">
     161        <table style="width: 100%; border-collapse: collapse;">
    159162        <%
    160163        int index = 0;
     
    185188            }
    186189            %>
    187             <tr id="<%=failureId%>" class="menuitem failure"
     190            <tr id="<%=failureId%>" class="failure highlight"
    188191              onclick="<%=onClick%>"
    189192              title="<%=tooltip%>"
    190193              >
    191               <td width="20px" class="index text"><%=index%></td>
    192               <td width="20px" class="text"><base:icon image="<%=severity == Severity.ERROR ? "error.png" : "warning.png"%>" /></td>
    193               <td class="text"><%=HTML.encodeTags(failure.getMessage()) %></td>
     194              <td style="width: 3em;" class="index"><%=index%></td>
     195              <td style="width: 24px; text-align: center;" class="text"><base:icon image="<%=severity == Severity.ERROR ? "error.png" : "warning.png"%>" /></td>
     196              <td class="text link"><%=HTML.encodeTags(failure.getMessage()) %></td>
    194197              <td class="text"><%=HTML.encodeTags(node.getTitle()) %></td>
    195198            </tr>
  • trunk/www/common/overview/info.jsp

    r5951 r5957  
    9494      if (subtype != null)
    9595      {
    96         path += " <span class=\"subtype\">(" + Base.getLinkedName(ID, subtype, false, true) + ")</span>";
     96        path += " <span class=\"itemsubtype\">(" + Base.getLinkedName(ID, subtype, false, true) + ")</span>";
    9797      }
    9898    }
     
    109109{
    110110  final GenericOverview overview = OverviewUtil.getCurrentOverview(sc);
     111  final ItemContext cc = sc.getCurrentContext(overview.getRootItem().getType());
    111112  final SnapshotManager snapshotManager = new SnapshotManager();
    112113  final AnnotationTypeFilter annotationTypeFilter = new AnnotationTypeFilter();
     
    123124  %>
    124125  <base:page title="" type="iframe">
    125   <base:head styles="newjoust.css">
     126  <base:head styles="table.css">
    126127  <script language="JavaScript">
    127128  function editItem(itemType, itemId, extraUrl)
     
    136137    url += '&annotationtype_id='+annotationTypeId;
    137138    url += '&standalone=1';
    138     Main.openPopup(url, 'EditAnnotation', 700, 460);
     139    Main.openPopup(url, 'EditAnnotation', 750, 500);
    139140  }
    140141  function editInheritedAnnotation(itemType, itemId, annotationTypeId)
     
    145146    url += '&annotationtype_id='+annotationTypeId;
    146147    url += '&standalone=1';
    147     Main.openPopup(url, 'EditAnnotation', 700, 460);
     148    Main.openPopup(url, 'EditAnnotation', 750, 500);
    148149  }
    149150  function editDataFile(itemType, itemId)
     
    154155  </script>
    155156  <style>
    156   .subtype, .subtype .link {
    157     font-size: 10px;
     157  .itemsubtype .link
     158  {
    158159    color: #777777;
    159160  }
     
    281282                {
    282283                  %>
    283                   <span class="subtype">(<%=Base.getLinkedName(ID, subtype, false, true)%>)</span>
     284                  <span class="itemsubtype">(<%=Base.getLinkedName(ID, subtype, false, true)%>)</span>
    284285                  <%
    285286                }
     
    406407      {
    407408        %>
    408         <h4><%=annotationsTitle%></h4>
    409         <table border=0 cellspacing=0 cellpadding=0>
    410         <%
    411         for (AnnotationSnapshot snapshot : annotations)
    412         {
    413           Annotation a = snapshot.getAnnotation(dc);
    414           String unitSymbol = a.hasUnit() ? "&nbsp;" + a.getUnitSymbol(null) : "";
    415           AnnotationType at = null;
    416           Annotatable annotationOwner = null;
    417           try
    418           {
    419             at = snapshot.getAnnotationType(dc);
    420           }
    421           catch (Throwable t)
    422           {}
    423           try
    424           {
    425             annotationOwner = snapshot.getItem(dc);
    426           }
    427           catch (Throwable t)
    428           {}
    429           String icon = at != null && at.isProtocolParameter() ?
    430               "parameter.png" : "joust/annotation.gif";
    431           String editLink = null;
    432           String editTooltip = null;
    433           if (annotatable != null && annotatable.hasPermission(Permission.WRITE) && at != null)
    434           {
    435             if (annotatable.equals(annotationOwner))
    436             {
    437               editLink = "editAnnotation('" + annotatable.getType().name() + "', " + annotatable.getId() + ", " + at.getId() + ")";
    438               editTooltip = "Edit the values of this annotation";
    439             }
    440             else
    441             {
    442               editLink = "editInheritedAnnotation('" + annotatable.getType().name() + "', " + annotatable.getId() + ", " + at.getId() + ")";
    443               editTooltip = "Inherit this annotation from another item";
    444             }
    445           }
    446           %>
    447           <tr>
    448             <td><base:icon visible="<%=icon != null%>" image="<%=icon%>" />&nbsp;</td>
    449             <td>
     409        <base:section
     410          id="allAnnotations"
     411          title="<%="Annotations &amp; parameters (" +  annotations.size() + ")"%>"
     412          context="<%=cc %>">
     413       
     414        <tbl:table id="annotations">
     415          <tbl:columndef id="annotation" title="Annotation/parameter" />
     416          <tbl:columndef id="owner" title="Owner" />
     417          <tbl:columndef id="values" title="Value" />
     418          <tbl:columndef id="edit" title="" />
     419          <tbl:data>
     420            <tbl:headers>
     421              <tbl:headerrow>
     422                <tbl:columnheaders />
     423              </tbl:headerrow>
     424            </tbl:headers>
     425            <tbl:rows>
    450426              <%
    451               if (at != null)
     427              for (AnnotationSnapshot snapshot : annotations)
    452428              {
     429                Annotation a = snapshot.getAnnotation(dc);
     430                String unitSymbol = a.hasUnit() ? "&nbsp;" + a.getUnitSymbol(null) : "";
     431                AnnotationType at = null;
     432                Annotatable annotationOwner = null;
     433                try
     434                {
     435                  at = snapshot.getAnnotationType(dc);
     436                }
     437                catch (Throwable t)
     438                {}
     439                try
     440                {
     441                  annotationOwner = snapshot.getItem(dc);
     442                }
     443                catch (Throwable t)
     444                {}
     445                String icon = at != null && at.isProtocolParameter() ?
     446                    "parameter.png" : "joust/annotation.gif";
     447                String editLink = null;
     448                String editTooltip = null;
     449                if (annotatable != null && annotatable.hasPermission(Permission.WRITE) && at != null)
     450                {
     451                  if (annotatable.equals(annotationOwner))
     452                  {
     453                    editLink = "editAnnotation('" + annotatable.getType().name() + "', " + annotatable.getId() + ", " + at.getId() + ")";
     454                    editTooltip = "Edit the values of this annotation";
     455                  }
     456                  else
     457                  {
     458                    editLink = "editInheritedAnnotation('" + annotatable.getType().name() + "', " + annotatable.getId() + ", " + at.getId() + ")";
     459                    editTooltip = "Inherit this annotation from another item";
     460                  }
     461                }
    453462                %>
    454                 <%=Base.getLinkedName(ID, at, false, true)%>
    455                 <%
    456               }
    457               else
    458               {
    459                 %>
    460                 <%=a.getValueType() %>
     463                <tbl:row>
     464                  <tbl:cell column="annotation"><base:icon visible="<%=icon != null%>" image="<%=icon%>" />
     465                    <%
     466                    if (at != null)
     467                    {
     468                      %>
     469                      <%=Base.getLinkedName(ID, at, false, true)%>
     470                      <%
     471                    }
     472                    else
     473                    {
     474                      %>
     475                      <%=a.getValueType() %>
     476                      <%
     477                    }
     478                    %>
     479                  </tbl:cell>
     480                  <tbl:cell column="owner">
     481                    <%
     482                    if (annotationOwner != null && !annotationOwner.equals(annotatable))
     483                    {
     484                      String ownerName = annotationOwner instanceof Nameable ?
     485                          ((Nameable)annotationOwner).getName() : annotationOwner.toString();
     486                      %>
     487                      <%=Base.getLink(ID, ownerName, annotationOwner.getType(), annotationOwner.getId(), annotationOwner.hasPermission(Permission.WRITE)) %>
     488                      <%
     489                    }
     490                    %>
     491                  </tbl:cell>
     492                  <tbl:cell column="values"><%=Values.getString(a.getValues(null), ", ", true)%><%=unitSymbol%></tbl:cell>
     493                  <tbl:cell column="edit"><base:icon image="edit.png" onclick="<%=editLink%>"
     494                    tooltip="<%=editTooltip%>" visible="<%=editLink != null %>"/></tbl:cell>
     495                </tbl:row>
    461496                <%
    462497              }
    463498              %>
    464             </td>
    465             <td>
    466               <%
    467               if (annotationOwner != null && !annotationOwner.equals(annotatable))
    468               {
    469                 String ownerName = annotationOwner instanceof Nameable ?
    470                     ((Nameable)annotationOwner).getName() : annotationOwner.toString();
    471                 %>
    472                 <base:icon image="bullet.png" tooltip="Annotation is inherited from" />
    473                 <%=Base.getLink(ID, ownerName, annotationOwner.getType(), annotationOwner.getId(), annotationOwner.hasPermission(Permission.WRITE)) %>
    474                 <%
    475               }
    476               %>
    477             </td>
    478             <td><base:icon image="bullet.png" /></td>
    479             <td><%=Values.getString(a.getValues(null), ", ", true)%><%=unitSymbol%></td>
    480             <td>&nbsp;
    481               <%
    482               if (editLink != null)
    483               {
    484                 %><base:icon image="edit.png" onclick="<%=editLink%>"
    485                   tooltip="<%=editTooltip%>" />
    486                 <%
    487               }
    488               %>
    489             </td>
    490           </tr>
    491           <%
    492         }
    493         %>
    494         </table>
     499            </tbl:rows>
     500          </tbl:data>       
     501        </tbl:table>
     502        </base:section>
    495503        <%
    496504      }     
     
    498506      {
    499507        %>
    500         <h4>Failure details</h4>     
     508        <base:section
     509          id="failures"
     510          title="<%="Failure details (" + (node.getNumErrors() + node.getNumWarnings()) + ")"%>"
     511          context="<%=cc %>"
     512          >
    501513        <table>
    502514        <%
     
    512524            %>
    513525            <tr>
    514               <td><base:icon image="<%=severity == Severity.ERROR ?
    515                 "error.png" : "warning.png"%>" /></td>
     526              <td><base:icon image="<%=severity == Severity.ERROR ? "error.png" : "warning.png"%>" style="padding-right: 4px;"/></td>
    516527              <td><b><%=HTML.encodeTags(failure.getMessage())%></b></td>
    517528            </tr>
    518529            <tr>
    519               <td style="border-bottom: 1px dotted #999999;"></td>
    520               <td style="border-bottom: 1px dotted #999999;">
     530              <td></td>
     531              <td style="padding-bottom: 0.3em;">
    521532                <%=HTML.encodeTags(validator.getDescription())%><br>
    522533                <%
     
    587598        {
    588599          %>
     600          <div class="messagecontainer note">
    589601          No validation failures on this item
     602          </div>
    590603          <%
    591         }       
     604        }
     605        %>
     606        </base:section>
     607        <%
    592608      }
    593609      %>
  • trunk/www/common/overview/tree.jsp

    r5951 r5957  
    218218    IconStore.addIcon('Warning', getRoot() + 'images/warning.png', 16, 16);
    219219    IconStore.addIcon('Error', getRoot() + 'images/error.png', 16, 16);
    220     IconStore.addIcon('FolderWarning', path + 'folderwarning.png', 18, 16);
     220    IconStore.addIcon('FolderWarning', path + 'folderwarning.gif', 18, 16);
    221221    IconStore.addIcon('FolderWarningSelected', path + 'folderwarningselected.gif', 18, 16);
    222     IconStore.addIcon('FolderChildWarning', path + 'folderchildwarning.png', 18, 16);
     222    IconStore.addIcon('FolderChildWarning', path + 'folderchildwarning.gif', 18, 16);
    223223    IconStore.addIcon('FolderChildWarningSelected', path + 'folderchildwarningselected.gif', 18, 16);
    224224    IconStore.addIcon('FolderError', path + 'foldererror.gif', 18, 16);
     
    228228    IconStore.addIcon('Item', path + 'item.gif', 18, 16);
    229229    IconStore.addIcon('ItemSelected', path + 'itemselected.gif', 18, 16);
    230     IconStore.addIcon('ItemWarning', path + 'itemwarning.png', 18, 16);
     230    IconStore.addIcon('ItemWarning', path + 'itemwarning.gif', 18, 16);
    231231    IconStore.addIcon('ItemWarningSelected', path + 'itemwarningselected.gif', 18, 16);
    232     IconStore.addIcon('ItemChildWarning', path + 'itemchildwarning.png', 18, 16);
     232    IconStore.addIcon('ItemChildWarning', path + 'itemchildwarning.gif', 18, 16);
    233233    IconStore.addIcon('ItemChildWarningSelected', path + 'itemchildwarningselected.gif', 18, 16);
    234234    IconStore.addIcon('ItemError', path + 'itemerror.gif', 18, 16);
Note: See TracChangeset for help on using the changeset viewer.