ClioSport.net

Register a free account today to become a member!
Once signed in, you'll be able to participate on this site by adding your own topics and posts, as well as connect with other members through your own private inbox!

  • When you purchase through links on our site, we may earn an affiliate commission. Read more here.

website feedback



  Ford Focus ST2
ordered about 1 1/2 hours ago :p

just applying the DNS settings should be live in the next 24 / 48 hours
 
  306 TD Slut
the list down the right hand side with the green ticks could do with a bigger line height. just to space it out a bit
 
  R26
I agree with furry_cup in that you could do with spreading the list to the right out a bit more. I also noticed you have it as a paragraph seperated by <br /> tags. Might be better if you did it as an unordered list and use css to replace the list-image with the green tick.

There also seems to be alot of space in the header on the right hand side you could use for something.

Otherwise it looks good, very clean and gets the message across.

-Jas.
 
  172
Firstly, why are using a frameset with only one frame? :S... and why are using framesets anyway?

Secondly, place this code in a file name of pngbehaviour.htc
Code:
<public:component>
<public:attach event="onpropertychange" onevent="propertyChanged()" />
<script>

var supported = /MSIE (5\.5)|[6789]/.test(navigator.userAgent) && navigator.platform == "Win32";
var realSrc;
var blankSrc = "/blank.gif";

if (supported) fixImage();

function propertyChanged() {
   if (!supported) return;

   var pName = event.propertyName;
   if (pName != "src") return;
   // if not set to blank
   if ( ! new RegExp(blankSrc).test(src))
      fixImage();
};

function fixImage() {
   // get src
   var src = element.src;

   // check for real change
   if (src == realSrc) {
      element.src = blankSrc;
      return;
   }

   if ( ! new RegExp(blankSrc).test(src)) {
      // backup old src
      realSrc = src;
   }

   // test for png
   if ( /\.png$/.test( realSrc.toLowerCase() ) ) {
      // set blank image
      element.src = blankSrc;
      // set filter
      element.runtimeStyle.filter = "progid:DXImageTransform.Microsoft.AlphaImageLoader(src='" +
                                     src + "',sizingMethod='scale')";
   }
   else {
      // remove filter
      element.runtimeStyle.filter = "";
   }
}

</script>
</public:component>
Make a 1px by 1px transparent gif and call it blank.gif, then add this to your head block
Code:
<!--[if IE 6]>
<style type="text/css" media="screen">
img { behavior: url("/pngbehaviour.htc"); }
</style>
<![endif]-->
You may need to change the paths etc, but it will make your 24-bit transparent pngs, transparent in IE6 (providing the user hasn't disabled JS.

Thirdly, get rid of the clock.
 

sn00p

ClioSport Club Member
  A blue one.
Firstly, why are using a frameset with only one frame? :S... and why are using framesets anyway?

Secondly, place this code in a file name of pngbehaviour.htc
Code:
<public:component>
<public:attach event="onpropertychange" onevent="propertyChanged()" />
<script>

var supported = /MSIE (5\.5)|[6789]/.test(navigator.userAgent) && navigator.platform == "Win32";
var realSrc;
var blankSrc = "/blank.gif";

if (supported) fixImage();

function propertyChanged() {
   if (!supported) return;

   var pName = event.propertyName;
   if (pName != "src") return;
   // if not set to blank
   if ( ! new RegExp(blankSrc).test(src))
      fixImage();
};

function fixImage() {
   // get src
   var src = element.src;

   // check for real change
   if (src == realSrc) {
      element.src = blankSrc;
      return;
   }

   if ( ! new RegExp(blankSrc).test(src)) {
      // backup old src
      realSrc = src;
   }

   // test for png
   if ( /\.png$/.test( realSrc.toLowerCase() ) ) {
      // set blank image
      element.src = blankSrc;
      // set filter
      element.runtimeStyle.filter = "progid:DXImageTransform.Microsoft.AlphaImageLoader(src='" +
                                     src + "',sizingMethod='scale')";
   }
   else {
      // remove filter
      element.runtimeStyle.filter = "";
   }
}

</script>
</public:component>
Make a 1px by 1px transparent gif and call it blank.gif, then add this to your head block
Code:
<!--[if IE 6]>
<style type="text/css" media="screen">
img { behavior: url("/pngbehaviour.htc"); }
</style>
<![endif]-->
You may need to change the paths etc, but it will make your 24-bit transparent pngs, transparent in IE6 (providing the user hasn't disabled JS.

Thirdly, get rid of the clock.

Depending on security settings, doesn't this pop up a security warning about the active-x control in ie?

Great code though!

We're not digging the clock are we?!
 
  172
When reviewing the code, I didn't see any comments about any issues with it being blocked, and the two machines at work I've tested it on previously (most likely default settings) did not complain.

It may affect some user if they've customised the security settings, I guess.

But I say, f**k 'em ;) lol

Oh, one more thing... change the style to this:
Code:
<!--[if lte IE 6]>
<style type="text/css" media="screen">
img { behavior: url("/pngbehaviour.htc"); }
</style>
<![endif]-->
 
  Ford Focus ST2
Firstly you well be happy to know the clock has gone :D

also its running on a frame due to way the domain is set up, this well be changing in the fix few days!

best to view it via http://www.rdmachinemaintenanceservices.co.uk/beta.php

can anyone test to see if the code above works on ie6?

There also seems to be alot of space in the header on the right hand side you could use for something.

Any ideas on what i can place here?

thanks
Robert
 
When reviewing the code, I didn't see any comments about any issues with it being blocked, and the two machines at work I've tested it on previously (most likely default settings) did not complain.

It may affect some user if they've customised the security settings, I guess.

But I say, f**k 'em ;) lol

Oh, one more thing... change the style to this:
Code:
<!--[if lte IE 6]>
<style type="text/css" media="screen">
img { behavior: url("/pngbehaviour.htc"); }
</style>
<![endif]-->

Stop. You're starting to look like you know what ya on about.. and it doesn't suit you. :rasp:
 
  R26
Looks fine in ie6... As for filling the gap why not put an image, but I'm not a designer so hopefully someone might have a good suggestion for what image...

-Jas.
 


Top