Advanced search  

News:

cpg1.5.48 Security release - upgrade mandatory!
The Coppermine development team is releasing a security update for Coppermine in order to counter a recently discovered vulnerability. It is important that all users who run version cpg1.5.46 or older update to this latest version as soon as possible.
[more]

Pages: [1]   Go Down

Author Topic: Error in Terra theme  (Read 2362 times)

0 Members and 1 Guest are viewing this topic.

rlhenson

  • Coppermine newbie
  • Offline Offline
  • Posts: 2
Error in Terra theme
« on: February 27, 2007, 11:55:50 pm »

I really like the Terra theme but noticed a minor error when using IE.
If you to to the demo In IE, right here at http://coppermine-gallery.net/demo/cpg14x/index.php?theme=terra you will see 

Coppermine Photo Gallery (Stable)
Demo for cpg1.4.x
-->

But if you use Firefox the --> is not there. I have looked all through the theme and can't find an extra --> with out an associated <!--

Does anyone know haw to fix this. I wish that everyone would just use Firefox but unfortunately IE is still the predominant browser.

Thanks in advance for any help,

Rick
« Last Edit: March 01, 2007, 07:44:21 pm by Nibbler »
Logged

Nibbler

  • Guest
Re: Error in Terra theme
« Reply #1 on: February 28, 2007, 12:01:51 am »

The script section at the top is not correct; the starting <script> tag is missing.

Code: [Select]
<script type="text/javascript" src="scripts.js"></script>
<!--
function MM_findObj(n, d) { //v4.01
  var p,i,x;  if(!d) d=document; if((p=n.indexOf("?"))>0&&parent.frames.length) {
    d=parent.frames[n.substring(p+1)].document; n=n.substring(0,p);}
  if(!(x=d[n])&&d.all) x=d.all[n]; for (i=0;!x&&i<d.forms.length;i++) x=d.forms[i][n];
  for(i=0;!x&&d.layers&&i<d.layers.length;i++) x=MM_findObj(n,d.layers[i].document);
  if(!x && d.getElementById) x=d.getElementById(n); return x;
}

function MM_showHideLayers() { //v6.0
  var i,p,v,obj,args=MM_showHideLayers.arguments;
  for (i=0; i<(args.length-2); i+=3) if ((obj=MM_findObj(args[i]))!=null) { v=args[i+2];
    if (obj.style) { obj=obj.style; v=(v=='show')?'visible':(v=='hide')?'hidden':v; }
    obj.visibility=v; }
}
//-->
</script>
Logged

Joachim Müller

  • Dev Team member
  • Coppermine addict
  • ****
  • Offline Offline
  • Gender: Male
  • Posts: 47843
  • aka "GauGau"
    • gaugau.de
Re: Error in Terra theme
« Reply #2 on: March 01, 2007, 07:50:45 am »

Just for clarification: the code posted by Nibbler needs to be changed to
Code: [Select]
<script type="text/javascript" src="scripts.js"></script>
<script type="text/javascript">
<!--
function MM_findObj(n, d) { //v4.01
  var p,i,x;  if(!d) d=document; if((p=n.indexOf("?"))>0&&parent.frames.length) {
    d=parent.frames[n.substring(p+1)].document; n=n.substring(0,p);}
  if(!(x=d[n])&&d.all) x=d.all[n]; for (i=0;!x&&i<d.forms.length;i++) x=d.forms[i][n];
  for(i=0;!x&&d.layers&&i<d.layers.length;i++) x=MM_findObj(n,d.layers[i].document);
  if(!x && d.getElementById) x=d.getElementById(n); return x;
}

function MM_showHideLayers() { //v6.0
  var i,p,v,obj,args=MM_showHideLayers.arguments;
  for (i=0; i<(args.length-2); i+=3) if ((obj=MM_findObj(args[i]))!=null) { v=args[i+2];
    if (obj.style) { obj=obj.style; v=(v=='show')?'visible':(v=='hide')?'hidden':v; }
    obj.visibility=v; }
}
//-->
</script>
Logged

rlhenson

  • Coppermine newbie
  • Offline Offline
  • Posts: 2
Re: Error in Terra theme
« Reply #3 on: March 01, 2007, 07:05:18 pm »

Thank you very much for both responses. that did the trick.

Rick
Logged
Pages: [1]   Go Up
 

Page created in 0.042 seconds with 15 queries.