View Full Version : Please Critique/Review my website
layyous
12-25-06, 09:43 AM
http://www.layyous.com/
Hi all,
I've hosted sites at powweb for five years . My web-development is completely self taught and I've used 'Frontpage' exclusively and adjusted the code using HTML validators.
I have tried to reach the international community although using automatic translators has it's short comings.I am working on improving the translation.
I will be gratefull for your constructive Critique/Review.
Thank you in advance
http://www.layyous.com/
Choosing many languages throws up a script error around line 58 or 57. This would be enough for many to leave!
layyous
12-26-06, 02:26 PM
Choosing many languages throws up a script error around line 58 or 57. This would be enough for many to leave!
Thank you .the cause of script error is
<body onload="layyous=window.open('http://www.layyous.com/good_bye.htm','popup','width=290,height=360,left=0 ,top=0');layyous.window.blur()">
which pop under a good bye message.when opening more than one page containing this script, it sends an script error message.
I can not figure out how to rectify this and will be gratefull for your help
layyous
12-26-06, 02:45 PM
Start with your top page (http://validator.w3.org/check?verbose=1&uri=http%3A%2F%2Fwww.layyous.com%2F).
Thank you,I fixed some of the errors.but I am left with some that I do not know what to do with namely:
-This page is not Valid HTML 4.01 Transitional!?? I tried other document types and got same error message.
-<form name="messages" onsubmit="return false">
the error is in the form action,so i Tried to put form action="" and wonder if it is the right way.
-also <input type="text" name="field" size="60" value onfocus="self.status='This is a JavaScript info. field'; return true" onchange="nochange()" style="font-family: Arial, serif, sans-serif; font-size: 18pt; font-weight:bold; text-align:center">
gives me error in :The "value" attribute needs a value. and do not know how to solve it.
-in table the attribute "HEIGHT".gives error ??? what to do
-another problem when I put a script to go back a page
<a href="#" onClick="history.go(-1)"><img border="0" src="../images/back_button.jpg" alt="<-<-<----" width="50" height="46"></a>
it gives an error : The "href" attribute specifies an internal link that you may not want. An internal link name should usually follow the # character, but it doesn't in this case.
I will gratefull for your help in solving these problems
Thank you .the cause of script error is
<body onload="layyous=window.open('http://www.layyous.com/good_bye.htm','popup','width=290,height=360,left=0 ,top=0');layyous.window.blur()">
which pop under a good bye message.when opening more than one page containing this script, it sends an script error message.
I can not figure out how to rectify this and will be gratefull for your helpCould this be caused by a pop-up blocker rather than a script error? Many people now block pop-up/pop-unders, me included.
layyous
12-26-06, 03:16 PM
Could this be caused by a pop-up blocker rather than a script error? Many people now block pop-up/pop-unders, me included.
No it is not caused by a pop-up blocker.I turned any pop up blocker off and still get the same script error message.
Builder
12-26-06, 05:03 PM
-another problem when I put a script to go back a page
<a href="#" onClick="history.go(-1)"><img border="0" src="../images/back_button.jpg" alt="<-<-<----" width="50" height="46"></a>
it gives an error : The "href" attribute specifies an internal link that you may not want. An internal link name should usually follow the # character, but it doesn't in this case.
Try this instead:
<a href="javascript:history.back()"><img border="0" src="../images/back_button.jpg" alt="<-<-<----" width="50" height="46"></a>
Kevin
layyous
12-26-06, 05:42 PM
Try this instead:
<a href="javascript:history.back()"><img border="0" src="../images/back_button.jpg" alt="<-<-<----" width="50" height="46"></a>
Kevin
Thank you very much.it worked .One problem solved.
Thank you again
layyous
12-27-06, 01:53 AM
You'll continue to get this error until all your errors are fixed.
That'll work.
Do the same as above... value=""
You cannot use the height attribute within the table tag. Let the content control the height of the table.
Thank you for your help.great,more problems fixed.
the code for loading flash files also gives load of errors
<OBJECT width="359" height="325" classid="clsid:D27CDB6E-AE6D-11cf-96B8-444553540000" name="ectopic_player.swf" id="ectopic_player.swf" codebase="http://download.macromedia.com/pub/shockwave/cabs/flash/swflash.cab#version=7,0,0,0">
<PARAM name="movie" value="../videos/ectopic_player.swf">
<PARAM name="loop" value="false">
<PARAM name="quality" value="high">
<PARAM name="play" value="TRUE">
<EMBED width="359" height="325" src="../videos/ectopic_player.swf" name="ectopic_player.swf" loop="false" type="application/x-shockwave-flash" pluginspage="http://www.macromedia.com/shockwave/download/index.cgi?P1_Prod_Version=ShockwaveFlash"></EMBED>
</OBJECT>
I will be gratefull for your help
thank you again
Builder
12-27-06, 09:56 PM
I think it's the <EMBED> tag causing the issues there. EMBED was deprecated by W3C 6, 7, 8 years ago, but was needed to accommodate Netscape 3 or 4. If you don't care about supporting 8-10 year old browsers that are now used only by a fraction of 1% of your visitors, you can safely delete it from your code. But test it first on newer browsers to be sure. :)
Good luck,
Kevin
Kitchensink108
12-27-06, 11:47 PM
The design isn't amazing but it's bearable. You use too many animated gifs and low quality images, though. On the awards pages, get rid of anything that has to do with only the website (golden web, world website, etc). These sites give out their "awards" to anybody who asks, plus, it makes your site look bad for several reasons.
Stop trying to throw up a pop-up window every time I view/leave a page or whatever.
If you can't get someone to translate the site manually, don't include a version in that language. Automatic translators are low quality, and reading text through a translation would be like reading broken English for us (a lot of bad grammar, unnatural sentences, etc) which reflect poorly on the site as a whole. I don't know enough German to critique translations themselves, though. It would be better to skip the "language select" page and go right to the English homepage if your audience mainly speaks English.
I could go farther, but those are the only big issues I found.
YvetteKuhns
12-28-06, 01:59 PM
While the guys pick on your source code, I will look at the actual web page as a visitor would. The first page shows a generic cartoonish image of a woman and baby in big hands. As a Christian, I would think of a mother and child in God's hands, though the hands could be of a doctor. A photo of a mother and baby would be better.
The first page is not in English, so I would click the English link if interested, but I am not. For the review, I will, but the first page did not give enough information or look interesting enough to keep me interested. This website could be about single mothers and I am not one. It could be about financial or spiritual help for mothers, adoption or any number of things.
The first page should define the target audience which means it should say who provides what product, service or information to whom and where. Do you provide services to woman where I am? I can't tell from here. You could use Javascript to provide the English version upon MouseOver or clicking the English button.
Now that I have clicked the English link, I feel overwhelmed with text, link choices and information. The sections of the website can be explained under each category and subcategory in the links menu or site map. If I want to see Ultrasound contents, I would click a link to Ultrasound. This English version of a home page should simply say that Dr Najeeb Layyous, OB GYN, is an infertility specialist who has assisted in reproductive technologies (conventional in vitro fertilization, intracytoplasmic sperm injection and other techniques) to help women in (designated location, Jordan?) get pregnant.
Simplify the website. Use professional photos instead of generic clip art. Do not use the pointing finger animation. The website has a lot of useful information, but getting to those pages means going through the intro page and the overwhelming, unwelcoming index page. People don't want to scroll too much to find what they want to read. Try to keep things in the first or second screen view.
The contact page does tell me where the doctor is, but not where the doctor works. Some doctors will travel, so I am still unsure of the target audience location (within so many kilometers of a specific city or region). The envelope animation is not necessary. Animations are distracting. There should be a logo or the Doctor's name, specialty and location at the top of every page.
There are other things like bookmarks or making this page the home page that are not necessary. If people like the site, they know how to use the browser to do this. I would suggest the recommend this site to a friend script, though. One more thing, the intro page buttons do not fit with the site. Try something else that would give a maternal feel or professional feel. Your target audience should be married couples or women who want to have children. What works for your location? Ask women there.
Here is a randomly picked example found AFTER my review, but it seems to fit what I was saying! Please see http://www.fertilitydr.com and notice that the color scheme is often blue and white for health-related websites, at least in the United States. After some research, I see that Europe and the Middle East have also used that color scheme.
I have a nephew whose wife has been trying to get pregnant with the help of fertility specialists. I had no trouble getting pregnant, but I had a few miscarriages. I am lucky to have one son. This is a very important website, so I hope you can make it work to help people. Good luck!
layyous
12-29-06, 01:26 PM
Thank you Scooter McDue for taking the time to review my site,I have fixed the popunder window and will work to fix the other issues.in the mean time if you can think of other points ,I will be gratefull
layyous
12-29-06, 01:39 PM
While the guys pick on your source code, I will look at the actual web page as a visitor would. The first page shows a generic cartoonish image of a woman and baby in big hands. As a Christian, I would think of a mother and child in God's hands, though the hands could be of a doctor. A photo of a mother and baby would be better.
The first page is not in English, so I would click the English link if interested, but I am not. For the review, I will, but the first page did not give enough information or look interesting enough to keep me interested. This website could be about single mothers and I am not one. It could be about financial or spiritual help for mothers, adoption or any number of things.
The first page should define the target audience which means it should say who provides what product, service or information to whom and where. Do you provide services to woman where I am? I can't tell from here. You could use Javascript to provide the English version upon MouseOver or clicking the English button.
Now that I have clicked the English link, I feel overwhelmed with text, link choices and information. The sections of the website can be explained under each category and subcategory in the links menu or site map. If I want to see Ultrasound contents, I would click a link to Ultrasound. This English version of a home page should simply say that Dr Najeeb Layyous, OB GYN, is an infertility specialist who has assisted in reproductive technologies (conventional in vitro fertilization, intracytoplasmic sperm injection and other techniques) to help women in (designated location, Jordan?) get pregnant.
Simplify the website. Use professional photos instead of generic clip art. Do not use the pointing finger animation. The website has a lot of useful information, but getting to those pages means going through the intro page and the overwhelming, unwelcoming index page. People don't want to scroll too much to find what they want to read. Try to keep things in the first or second screen view.
The contact page does tell me where the doctor is, but not where the doctor works. Some doctors will travel, so I am still unsure of the target audience location (within so many kilometers of a specific city or region). The envelope animation is not necessary. Animations are distracting. There should be a logo or the Doctor's name, specialty and location at the top of every page.
There are other things like bookmarks or making this page the home page that are not necessary. If people like the site, they know how to use the browser to do this. I would suggest the recommend this site to a friend script, though. One more thing, the intro page buttons do not fit with the site. Try something else that would give a maternal feel or professional feel. Your target audience should be married couples or women who want to have children. What works for your location? Ask women there.
Here is a randomly picked example found AFTER my review, but it seems to fit what I was saying! Please see http://www.fertilitydr.com and notice that the color scheme is often blue and white for health-related websites, at least in the United States. After some research, I see that Europe and the Middle East have also used that color scheme.
I have a nephew whose wife has been trying to get pregnant with the help of fertility specialists. I had no trouble getting pregnant, but I had a few miscarriages. I am lucky to have one son. This is a very important website, so I hope you can make it work to help people. Good luck!
Thank you Yvette for taking the time to review my site.You seem to be an Expert in web design.Your points are valid.You have left me with a lot of work to fix some issues,and scratching my head about The first page.
-I got rid of bookmarking or making this page the home page
-I got rid of the pointing finger animation
-I got rid of the The envelope animation
I will work to implement your suggestions,but in the mean time ,if you have more of your wonderfull comments,I will be gratefull
YvetteKuhns
1-10-07, 03:27 PM
You have improved your English index page by making it less cluttered and fitting contents in the first screen view. But the text still needs to be more concise. As I said before, you must define your target audience. You do not have to make personal comments here. You may do so elsewhere if you wish.
Thank you for visiting my site which I set up hoping that I will be able to fulfill my aim of conveying my long clinical and practical experience for the public benefit. This award-winning site is regarded as one of the largest and most important medical sites in the middle east.
The site provides simplified descriptions of medical topics relating to gynaecology, pregnancy, female and male infertility, assisted reproductive technologies and Ultrasonography, that are also portrayed in the form of pictures and video clips.
We are always updating this site with the latest medical advances that we follow in our medical centre located in Amman, Jordan using the latest technology (see available treatments).
To reach more people, I translated this site to as many languages as possible, although English and Arabic will remain the primary languages on the site. Thus trying to achieve my goal in educating as many people around the world as possible.
This must be edited.
Dr. Najeeb Layyous is an infertility doctor practicing in a medical centre located in Amman, Jordan. This medical website provides simplified descriptions, pictures and video clips of medical topics relating to gynecology, pregnancy, female and male infertility, assisted reproductive technologies and Ultrasonography.
You can edit this paragraph, but try to stay with this. Don't add anything too personal or unimportant here. You don't have to thank people for visiting, because they know you want them to visit your website. You don't have to explain why you have multiple language; that is also obvious.
The FAQ page is long, the text is bold and large and there should be a Back to Top of Page link at the bottom and possibly after each question. Try using smaller text and simply use bold text for the question and not the answer. Or use another text color. But don't use blue text or underlined text, if the text is not a link. That can confuse readers.
The pregnancy video clips page has an unnecessary animation. Under it, it says "Press on the picture to view movie" where you want someone to select an image from the choices, but some people may try to click that animation!
I just noticed "Music to Enjoy" where you may need to remove that or replace songs with those you have permission to use. While your website is educational, you may still need permission to include copyrighted music. The songs do sound like the type you buy for websites and other projects that require music. The music reminds me of the waiting room of a doctor's office or dentist.
The construction of the website page is full of goofy animations that are not appropriate for a professional, educational, medical website. Do NOT link to your webstats from your website! That information should be protected. You probably have plenty of other contents that should be edited or removed. Many people put links on their websites for their own convenience. If the contents are not for the public, do not link to them.
Links to pages of your website should open in the same window. Links to other websites should open in a new window. I had a bunch of windows open after clicking links. That can be annoying. Also include your header (logo), navigation (links), and footer (copyright info) on EVERY page. That includes your Tell a Friend page. Instead of using Bravenet's stuff, you can install free scripts on your server.
Because you used Front Page, you have file/folder names that work for Microsoft but not other systems. For example, root folder should be rootfolder or else it appears as root%20folder and may appear as a broken link. You can use the underscore for root_folder.
Okay, I typed enough for now. Get back to work! ;)
Thank you Yvette for your help.you are great.I am working to implement your suggestions.In the mean time,if you think of other points I will be very grateful.
YvetteKuhns
1-12-07, 12:28 PM
Thank you Yvette for your help.you are great.
Thanks! My husband only says that after I undress. ;)
In the mean time,if you think of other points I will be very grateful.
Don't you have enough work?! :D Please post after making several changes and hopefully other people will make comments as well. I usually revisit posts when receiving email notification of a post, so any post will remind me to take another look.
Gamersea
1-25-07, 05:59 AM
Nice Site, However the Dutch section can be improved:P
YvetteKuhns
1-25-07, 10:38 AM
I suspect that any language translations that are not done by a person that knows both languages well will be flawed. That is why I don't try to speak in other languages. Many people use Babelfish or another translator, but I have seen examples that are incorrect. One word can change the meaning of the entire sentence.
Gamersea
1-25-07, 12:24 PM
Jep, Translator should only and then I mean only be used for translating 1 or 2 words. not for many lines of words.
Even then they don't always get it write (sic)
vBulletin v3.6.0, Copyright ©2000-2009, Jelsoft Enterprises Ltd.