Comment 2 for bug 633363

Revision history for this message
Dan Scott (denials) wrote :

To be honest, I don't understand why we would make this particular change in the default skin.

  * the link already has a class of "classic_link"; setting the id to "classic_link" seems strange
  * anyone running Evergreen in production will need to modify the skin, including the footer, and should blow this away if they're not in Georgia

However, I could see an argument for removing the link, and the corresponding buildEGGalLink() function in Open-ILS/web/opac/skin/default/js/sidebar.js, entirely, as it does add a small amount of attachEvt() overhead that should exist only in the PINES skin.

Chris, if you're reading this, would you be averse to pushing this functionality into the PINES-examples SVN repository?