www.TheHost.co.za - hospitality business site

[StefanRSA]

Hallo Guys!

I am VERY proud to say... I am (almost) done with my first WB site!!!!
Please do me a favour and send comments...

I am 3 months new in PhP and MySql, so if you pick up any problems please be so kind to tell me about it and give a hand if you can! Please note that the link without the /wb,  link to their old(UGLY!) site at
www,the-host,net.....

WB ROCKS!!!! MY NEW SITE ADDRESS!!!!! -----------------> http://www.thehost.co.za/wb 

[aldus]

Fine ... but there is an error in the "screen.css" in line 143:

Code: [Select]

background: #FFFFFF url(<?php echo TEMPLATE_DIR;?>/images/headcut_14-over.gif) repeat;
You can't place PHP-Code inside an external css file - will not be execute.
And you are not in the need to this: keep in mind, that all paths inside the css are rellative
to the css file, so in this case you can simple go like this:

Code: [Select]

background: #FFFFFF url(images/headcut_14-over.gif) repeat;
if there is a subfolder named "images" next to the css-file.

Regards
Aldus

[StefanRSA]

Fixed it, thanks Aldus!

[aldus]

Fine - nothing found yet
And (!) imagefloooooooooooooooooow woup

Regards
Aldus

[Stefek]

Nicely baked.
I like it.
Very complex template.
Looks professional!

I see you love to design every single detail. 

Regards,
Stefek

[StefanRSA]

Thanks Guys...

I have a problem in my menu.... Its a combination of JS and CSS conflict... Its for the active page menu color...(?submenuheader=1)
Don't have this problem if I go only CSS but I want the glide down effect of the menu...

[Stefek]

... Its a combination of JS and CSS conflict...

Please explain the exact problem.
For me it look just fine. (FireFox 2 / Win XP)

Regards,
Stefek

[StefanRSA]

The ?submenuheader=1 and ?submenuheader=0 works with my CSS to display the active page on the menu and also if the submenu should expand or not. This is the only way I can get the SUBMENUHEADER to also have a dif color when any sumenu page is selected.

The error shows in FF and IE in the background: Error: headers[expandedindex] is undefined
Source File: http://www.thehost.co.za/wb/pages/screen-shots.php?submenuheader=1
Line: 34

[Stefek]

Now I see.
Something wrong with your JS.
Can't help - sorry.

I'm certain that sombody else will help you to fix this.

Regards,
Stefek

[aldus]

As for a quick view: missing ";"
should be

Code: [Select]

for (var i=0; i<expandedindices.length; i++){
  var expandedindex=expandedindices[i]; //index of current expanded header index within array
  headers[expandedindex].style.backgroundColor='#00B8FF';
  headers[expandedindex].style.color='white';
 }

to avoid some unexpected results you should testing the results, e.g.

Code: [Select]

for (var i=0; i<expandedindices.length; i++){
  var expandedindex=expandedindices[i]; //index of current expanded header index within array
  if (expandedindex) {
    if (headers[expandedindex]) {
      headers[expandedindex].style.backgroundColor='#00B8FF';
      headers[expandedindex].style.color='white';
     }
   }
}

sorry - untested yet ...

Regards
Aldus

[StefanRSA]

That Solved my Problem, Thanks Aldus!

Thanks for you as well Stefek!!!