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?
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?