Ticket #608: ticket608.2.patch

File ticket608.2.patch, 1.3 KB (added by PhilHawksworth, 3 years ago)

Revised patch to implement Eric's solution as described at the top of this ticket

  • deprecated/Dom.js

     
    3535} 
    3636 
    3737// Remove a node and all it's children 
    38 function removeNode(e) 
    39 { 
     38function removeNode(e) { 
    4039        jQuery(e).remove(); 
    4140} 
    4241 
  • js/Lingo.js

     
    244244 
    245245merge(config.macros.toolbar,{ 
    246246        moreLabel: "more", 
    247         morePrompt: "Reveal further commands" 
     247        morePrompt: "Reveal further commands", 
     248        lessLabel: "less", 
     249        lessPrompt: "Hide further commands" 
    248250        }); 
    249251 
    250252merge(config.macros.refreshDisplay,{ 
  • js/Toolbar.js

     
    101101config.macros.toolbar.onClickMore = function(ev) 
    102102{ 
    103103        var e = this.nextSibling; 
    104         e.style.display = "inline"; 
    105         jQuery(this).remove(); 
     104        var showing=e.style.display=="block"; 
     105        e.style.display = showing?"none":"block"; 
     106        this.innerHTML=showing?config.macros.toolbar.moreLabel:config.macros.toolbar.lessLabel; 
     107        this.title=showing?config.macros.toolbar.morePrompt:config.macros.toolbar.lessPrompt; 
    106108        return false; 
    107109}; 
    108110