Changeset 4830


Ignore:
Timestamp:
Mar 23, 2009, 4:46:46 PM (13 years ago)
Author:
Martin Svensson
Message:

References #1240. Some adjustments on menuitems' tooltip- and cursor- behavior. The validate-toolbutton is decided to validate the whole tree and not only already loaded nodes.

Location:
trunk/www
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/www/common/overview/ajax.jsp

    r4823 r4830  
    2727  import="net.sf.basedb.util.overview.GenericOverview"
    2828  import="net.sf.basedb.util.overview.Node"
    29   import="net.sf.basedb.util.overview.OverviewUtil"
     29  import="net.sf.basedb.util.overview.OverviewUtil" 
    3030  import="net.sf.basedb.clients.web.Base"
    3131  import="net.sf.basedb.clients.web.WebException"
     32  import="net.sf.basedb.clients.web.util.HTML"
    3233%>
    3334<%
     
    6566      }
    6667      for (Node n : parent.getChildren())
    67       {       
     68      {
     69        String title;
    6870        String folderIcon = n.getNodeType() == Node.Type.FOLDER ? "Folder" : "Item";
    6971        if (n.getItemType() == Item.ANNOTATIONTYPE && parent.getItemType() == Item.PROTOCOL)
     
    8082        %>begin
    8183id  <%=n.getId()%>
    82 title <%=n.getTitle() + (n.getNodeType() == Node.Type.FOLDER ? " (" + numChildren + ")" : "")%>
     84title <%=HTML.javaScriptEncode(n.getTitle()) + (n.getNodeType() == Node.Type.FOLDER ? " (" + numChildren + ")" : "")%>
    8385hashCode  <%=n.hashCode()%>
    8486folderIcon  <%=folderIcon %>
    8587parentIconSuffix  <%=parentIconSuffix%>
    86 tooltip <%=tooltip%>
    87 parentTooltip <%=parentTooltip%>
     88tooltip <%=HTML.javaScriptEncode(tooltip)%>
     89parentTooltip <%=HTML.javaScriptEncode(parentTooltip)%>
    8890end
    8991<%
  • trunk/www/common/overview/overview.jsp

    r4767 r4830  
    122122        <tbl:button image="joust/minusonly.gif" title="Collapse all"
    123123          onclick="collapseAll()" tooltip="Collapse entire tree" />
    124         <tbl:button image="refresh.gif" title="Revalidate"
    125           onclick="revalidate()" tooltip="Revalidate the experiment" />
     124        <tbl:button image="refresh.gif" title="Validate"
     125          onclick="revalidate()" tooltip="Validate the whole tree" />
    126126        <tbl:button image="configure.png" title="Validation options&hellip;"
    127127          onclick="validationOptions()" tooltip="Change validation options" />
  • trunk/www/common/overview/tree.jsp

    r4823 r4830  
    228228
    229229  function lazyInitSubNode(nodeId)
    230   {
     230  {
     231    var parentMenu = JoustMenu.menuItems['N' + nodeId];
     232    JoustMenu.changeCursorOnMenuItem(parentMenu.index, 'progress');   
    231233    var request = Ajax.getXmlHttpRequest();
    232234    var url = 'ajax.jsp?ID=<%=ID%>&cmd=GetSubNodes&node_id=' + nodeId;
     
    237239    var kv = new Array();
    238240    var numAdded = 0;
    239     var parentMenu = JoustMenu.menuItems['N' + nodeId];
     241   
    240242    for (var i=0; i<response.length; i++)
    241243    {
     
    246248        var folderIcon = kv['folderIcon'];       
    247249        var parentIconSuffix = kv['parentIconSuffix'];
     250        var tooltip = kv['tooltip'] ? kv['tooltip'] : '';
     251        var parentTooltip = kv['parentTooltip'] ? kv['parentTooltip'] : '';                           
    248252        numAdded++;
    249253        var node = JoustMenu.addLazyChildItem(
    250254              parentMenu.index, folderIcon, kv['title'], 'showInfo(\''+id+'\')',
    251                 kv['tooltip'], id, 'lazyInitSubNode('+kv['hashCode']+')');
     255                tooltip , id, 'lazyInitSubNode('+kv['hashCode']+')');
    252256       
    253257        if (parentIconSuffix)
    254258        {
    255           changeParentIcons(parentMenu, parentIconSuffix, kv['parentTooltip']);         
    256           JoustMenu.menuItems['0'].iconName = parentIconSuffix;                             
    257           JoustMenu.updateIconsAndText(parentMenu.index, true);
     259          updateParentMenu(parentMenu, parentIconSuffix, parentTooltip);         
     260          JoustMenu.menuItems['0'].iconName = parentIconSuffix;
    258261          reloadSiblingFrames();
    259262        }
     
    264267        kv[tmp[0]] = tmp[1];
    265268      }
    266     }
     269      JoustMenu.updateIconsAndText(parentMenu.index, true);
     270    }   
     271    JoustMenu.changeCursorOnMenuItem(parentMenu.index, 'pointer');
    267272    return numAdded > 0;         
    268273  }
     
    274279  }
    275280
    276   function changeParentIcons(menuItem, iconSuffix, tooltip)
     281  function updateParentMenu(menuItem, iconSuffix, tooltip)
    277282  {
    278283    menuItem.tooltip = tooltip;
  • trunk/www/include/scripts/newjoust.js

    r4823 r4830  
    333333      }
    334334    }   
     335  }
     336 
     337  /**
     338    Change cursor on a menu item.
     339    @param menuItemIndex Index of the menu item over which the cursor should change.
     340    @param newCursor A string value of what to set the cursor to.
     341   */
     342  this.changeCursorOnMenuItem = function(menuItemIndex, newCursor)
     343  {
     344    var menuItem = this.menuItems[menuItemIndex];
     345    document.getElementById('menuItem'+menuItem.index).style.cursor=newCursor;
     346    document.getElementById('outlineIcon' + menuItem.index).style.cursor=newCursor;
     347    document.getElementById('docIcon' + menuItem.index).style.cursor=newCursor;
    335348  }
    336349}
Note: See TracChangeset for help on using the changeset viewer.