Pleas revies my first CSS Site

[faiztangi]

Hi,

It's my first review site and also my first site using CSS...
As I am new to webdesing and CSS, there will be lots of problum there in my site and I will appreciate the identificatio.

The url to my site is

http://rafaqatbabar.com


Thanks
Faiz

[Pcanywhere]

Hi, I actually have similar menu concept on my site. Only thing I don't really like on your site is your selection of colors and fonts.
Also its good to mark link on page that you are currently on (it ease navigation quite bit, vistor always know where he is, again like on my site ) )))

[ilyalyu]

I would advise to spell-check your site. For example, there is “IntroDCution” header on the introduction page. I believe people often judge about your professionalism by such mistakes.

Also, it might be useful to display menu item for the current page in a different color than others, so that the user can see on which page he is. For example, if you are on home page, then the “home” menu item should be in a different color. So, I know that there is no need to click this menu item because I am already on this page.

When you click “Check Mail” link (top right), the menu disappears and the background changes. Well, I believe that the structure of the page should be the same on all pages of the website. At least color scheme… It facilitates site navigation. Though the site becomes less colorful...

[faiztangi]

don't really like on your site is your selection of colors and fonts.

Advise me on that

Also its good to mark link on page that you are currently on (it ease navigation quite bit, vistor always know where he is, again like on my site

How to do that.....

Please give me your site url...

[faiztangi]

I would advise to spell-check your site.

Thanks for that

Also, it might be useful to display menu item for the current page in a different color than others, so that the user can see on which page he is. For example, if you are on home page, then the “home” menu item should be in a different color. So, I know that there is no need to click this menu item because I am already on this page.

How it can be done....

[ilyalyu]

How it can be done....

Well. First, it is not so critical in my view (I mean using diffirent color for menu item corresponding to the current page). Sometimes I do so, sometimes not.

It can be easily done using one of the web scripting languages, for example, PHP:

<?php

if ($_SERVER['PHP_SELF'] == '/page1.php')
echo 'Title for page1';
else echo '<a href="/page1.php">Title for page1</a>';

?>

I would not recomend doing it with javascript document.write() function, because such links will not be seen by search engines.

You can use links collection in javascript:

<script>

for (i = 0; i < document.links.length; i++)
if (document.links[0].href == document.location.href)
document.links[0].style.color = '#F00000';

</script>

This code does the folowing. If the link refrences current page it changes it color to red (sure you can choose another color). Try to put with code right after the your menu.

But again, the user might see the color changing. I am not sure whether it is good. So the best way is using server-side scripting langauge.

[gokeeffe]

For a first time effort it's quite good, I woul definitely spell check the site as it will look ameturish and also the background color I would prefer to see something a little bit bighter.

[faiztangi]

<script>

for (i = 0; i < document.links.length; i++)
if (document.links[0].href == document.location.href)
document.links[0].style.color = '#F00000';

</script>

This code does...

Please provide me full javascript code....I tried this code but it didnt worked for me...
I dont use Php but ASP...

[ilyalyu]

Please provide me full javascript code....I tried this code but it didnt worked for me...
I dont use Php but ASP...

I made a small mistake in javascript code.

It should be like that:

<script>

for (i = 0; i < document.links.length; i++)
  if (document.links[i].href == document.location.href)
    document.links[i].style.color = '#F00000';

</script>

Instead of

<script>

for (i = 0; i < document.links.length; i++)
  if (document.links[0].href == document.location.href)
    document.links[0].style.color = '#F00000';

</script>

And you should place it AFTER your links.

[Green Beast]

I'd kill that landing page. It looks awful and does nothing to get me to actually go to the real website... which isn't bad. If you review site stats, you'll find that most people will not travel beyond the first page unless they are there for specific content. Most people aren't going get past your seizure-inducing landing page. Not a good thing.

Mike

[faiztangi]

It should be like that:
....
And you should place it AFTER your links.

Yes that worked okayed....

Thanks for that....

Regards,

Faiz

[reedy]

Hi

I would say immediately do away with the opening splash page. Not only will most people skip it, the design you are then taken to looks completely different to the intro page. I had to check I was still on the same site.

How about adding a link titled 'Introduction Video' or something on the main site, so people can click it if they choose to view it?

The design of the actual site itself looks sober and professional, certainly fitting your line of business! Just to repeat an earlier posts; good spelling and grammar is essential and must be triple checked!

On a side note the domain name would be impossible for me (and I am guessing most Westerners) to remember. Could you come up with something else that would be easier to remember?

Best of luck with the site.

[jaspervd]

The intro looks nice, but doesnt fit your website.

Your main webpage is looking good, but a bit boring. Try to add some nice pictures or change the background color. I think changing the background color would help a lot.

[faiztangi]

I am agreed that the intro and the main page dont match...But the problum is that I cant change the intro I can only change the background color of the intro etc...
I also cant remove the intro as it's a must by my boss...
So I need suggestions how to make the intro and main page look compatabile...

Also if some one want me to change the backgroung color of the main page, what should be that...

[falloutsoldir]

I wouldnt have so much music in the intro, I always find music quite annoying unless used subtly, in only my opinion though

[MattJH]

The intro and main page are mismatched. The main site has elements which don't fit such as that welcome bar. The "Check Mail" thing isn't a link and so looks like a weird header. The text is built into the side images which is a bit bad practice and the links in the menu are in a font that doesn't match a clean style.

Change all that and i think it will look pretty good.

[faiztangi]

I think that the header in both the main page and the intro should be same ...
do any one agree if any one then i will change that...