Please Review! - Webmasters - Nairaland
Nairaland Forum / Science/Technology / Webmasters / Please Review! (673 Views)
Please Review This New Startup feature : site for creating forms / Please Review Www.enairablog.com / Please.... Review this site (2) (3) (4)
| Please Review! by Nobody: 3:51pm On Jul 27, 2013 |
Hello webmasters. I need you to please help me review http://www.justmeahphotography.com before i finally launch it. Awaiting your reviews. Thanks. |
| Re: Please Review! by greenmouse(m): 4:37pm On Jul 27, 2013 |
the best job you have ever done. but your code still look amature to me, may be @graphicplus that has time to review will expatiate more on this. But weldone, Nice design any way. YOUR HTML source code---Study it.... <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> <html xmlns="http://www.w3.org/1999/xhtml"> <head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8" /> <link type="text/css" rel="stylesheet" href="style.css" /> <link rel="stylesheet" href="styles/nivo-slider.css" type="text/css" media="screen" /> <link rel="stylesheet" href="styles/style2.css" type="text/css" media="screen" /> <script type="text/javascript" src="scripts/jquery-1.9.0.min.js"></script> <script type="text/javascript" src="scripts/jquery.nivo.slider.js"></script> <script type="text/javascript"> $(window).load(function() { $('#slider').nivoSlider(); }); </script> <title>JustMeah Photography | Get the SHOT, go the extreme</title> </head> <body> <div id="headerbg"> <div class="header"> </div><!--header--> <div class="wrap"> <img src="img/logo.png" alt="Justmeah Photography" class="logo" /> <div class="menu"> <ul> <li><a href="index.php">HOME</a></li> <li><a href="services.php">STUDIO SERVICES</a></li> <li><a href="gallery.php">PHOTO GALLERY</a></li> <li><a href="contact.php">CONTACT US</a></li> </ul> </div><!--menu--> </div><!--wrap--> </div><!--headerbg--> <div id="slide"> <div class="wrap"> <div class="slider-wrapper theme-default"> <div id="slider" class="nivoSlider"> <img src="images/slide.jpg" data-thumb="images/slide.jpg" alt="" /> <img src="images/slide2.jpg" data-thumb="images/slide2.jpg" alt=""/> <img src="images/slide3.jpg" data-thumb="images/slide3.jpg" alt="" /> <img src="images/slide4.jpg" data-thumb="images/slide4.jpg" alt="" /> <img src="images/slide5.jpg" data-thumb="images/slide5.jpg" alt="" /> <img src="images/slide6.jpg" data-thumb="images/slide6.jpg" alt="" /> </div> <div id="htmlcaption" class="nivo-html-caption"> <strong>This</strong> is an example of a <em>HTML</em> caption with <a href="#">a link</a>. </div> </div> </div><!--wrap--> </div><!--slide--> <div id="main"> <div class="box" > <h2>WELCOME</h2> <p> Welcome to the world of Digital Photography!<p> <p>Photography, unlike before, has now become a powerful tool today in the redefination our world. To so different people, different meaning... But to us, Photography is all.</p> <p>Just take your time to go through our online portfolio, you will discover the uniqueness of our photography - How we use our professional photography experience to document emotional reality of every event. There is no need to make your event photos look like that of anyone else. Every moment of your occasion is important to us and you truly deserve a personal creative story of your event. </p> <p>Don't hesitate calling us to help create a long lasting record of your next event using our out-of-the-box professional Photography experience</p> </div> <div class="box"> <h2>STUDIO SERVICES</h2> <p><img src="img/modelling.jpg" alt="" /></p> <p><img src="img/studioshots.jpg" alt="" /></p> <p><img src="img/weddings&events.jpg" alt="" /></p> </div> <div class="box"> <h2>RANDOM PHOTOSHOTS</h2> <img src="img/random/1_1mt.jpg" alt="" class="random"/> <img src="img/random/1_2mt.jpg" alt="" class="random"/> <img src="img/random/1_3mt.jpg" alt="" class="random"/> <img src="img/random/1_4mt.jpg" alt="" class="random"/> <img src="img/random/1_5mt.jpg" alt="" class="random"/> <img src="img/random/1_6mt.jpg" alt="" class="random"/> <img src="img/random/1_7mt.jpg" alt="" class="random"/> <img src="img/random/1_8mt.jpg" alt="" class="random"/> <div class="clear"></div> <br/><a href="gallery.html" style="float:right;margin-right:40px;">View more »</a><br/> <br/><br/> <h2>QUICK CONTACT</h2> <p>Navada Plaza, Liberty Stadium, Oke-Ado, Ibadan</p> <p>+234 (0) 803 3331976<br/>+234 (0) 809 8338080</p> <p>info@justmeahphotography.com</p> </div> <div class="clear"></div><!--clear--> </div><!--main--> <div id="footer"> <div class="wrap"> <p>© Justmeah Photography Studio - 2012. All Rights Reserved.</p> </div><!--wrap--> </div><!--footer--> </body> </html> |
| Re: Please Review! by Nobody: 4:42pm On Jul 27, 2013 |
greenmouse: the best job you have ever done.#smiles greenmouse:Ok. greenmouse:Thanks, bro |
| Re: Please Review! by GraphicsPlus(m): 4:44pm On Jul 27, 2013 |
Good job. Dark background is good for photography studio website. Take note of these: 1. Your logo should be clickable. It's good for seo. Just link the logo to index.php. Remember to add this: a img{border:0} in your css so that ie will not give it a border. 2. Tidy up of empty image alt. This is very important since this kind of website needs some traffic from search engines. 3. In your stylesheet, you did this: background-image:url('img/bg.jpg'); background-repeat:repeat; background-position:center; The background-repeat and background-position are not necessary in this case. Css repeats background image by default. So without telling it to repaet the background-image, it will repeat it. Unless you want to tell it to repeat-x or y. Again, since the image is seamless and will repeat till it fills up the whole body background, there is no need telling it to position center. You use background-positioning when the image is one and big, so in that case you will set the background-repeat to no-repeat and position it center. 4. Why did you position your logo and menu absolute. I cant see any reason for that. Whereas it worked out fine, but please try to be avoiding absolute positioning if you can. You should have just floated them left and right respectively and clear both. 5. I am not a fan of too much text-shadow. In your services.php page, the text may be better in just plain white color. The color you used to write "Weddinbg n events" is kind of not good for wedding. Why not use a vibrant color like the one u used to write "Modelling photoshots"? 6. There is no client-side form validation. Again, no thank you message. I filled a dummy data and sent and the form reloaded and returned without a thank you. This may make user not to be certain whether his query was sent or not. Again, instead of phone field, why not email. A user will be more willing to give his email than phone number. And again, it will be cheaper for the company to send a reply to the user via email than calling them and burning credits. 7. I think it will be good to set active link to show which page the user is viewing. Overall, good job. Your html and css codes are actually neat and properly indented. @Greenmouse, I dont think nothing is wrong with his html markup or is there? Although I can see he used multiple wrapper when he could have just used one. |
| Re: Please Review! by Nobody: 4:49pm On Jul 27, 2013 |
^^ Thank you so much, GraphicsPlus. All points well noted and proper adjustments will be made. Thanks once again. |
| Re: Please Review! by greenmouse(m): 4:50pm On Jul 27, 2013 |
GraphicsPlus: So you didn't notice that the HTML code is not SEO friendly and the use of multiply Heading tags, and many others, if na me now, you go sit down, do a screen shot, start teaching yourself lesson you may not need. #mtchhhhew |
| Re: Please Review! by Nobody: 4:55pm On Jul 27, 2013 |
Kai! Greenmouse, you dey quick vex oh |
| Re: Please Review! by GraphicsPlus(m): 4:56pm On Jul 27, 2013 |
@Greenmouse, just point out what you mean that his html is not seo friendly. We are here to learn. Talking about multiple heading tag, do you mean h2 and so on? Now, in seo, multiple heading tag is allowed. What is not seo friendly is having multiple h1 tag. The Op didnt even have a single h1 tag, which I forgot to point out. Every page should have one h1 tag and multiple other heading tags. So using multiple h2 tag is NOT a bad seo pratice. |
| Re: Please Review! by Nobody: 5:01pm On Jul 27, 2013 |
@GraphicPlus: Talking about having at least 1 h1 tag, where is it best to use such a h1 tag? NB: This is the first time i am hearing of such. Thanks |
| Re: Please Review! by GraphicsPlus(m): 5:57pm On Jul 27, 2013 |
yorme: @GraphicPlus: Ok. In the art of seo, it's one of the good seo practice. The h1 tag should be on top before other headings. Robots see texts in h1 as very important. Like in your home page, you may do something like this: <p>WELCOME</p> <h1> Welcome to JustMeah Photography - the world of Digital Photography!<h1> You may say, 'but my welcome text will now be small.' Here is the trick. Leave it like this in your html, then go to your css and change the font-sizes. Make the 'WELCOME' text to be big and then reduce the h1 tag font-size to normal p tag font-size of your page. Do same in all your pages. You can have multiple h2 to h6 headings, but restrict yourself to one h1 heading per page. |
| Re: Please Review! by Nobody: 6:41pm On Jul 27, 2013 |
Ok. Thank you. |
| Re: Please Review! by Djtm(m): 8:59pm On Jul 28, 2013 |
use a favicon. |
(1) (Reply)
Win 3 Free Pixelkit Premium Membership Accounts / Website Experts,your Opinion And Suggestions Will Be Highly Appreciated Here / Issue Resolved
(Go Up)
| Sections: politics (1) business autos (1) jobs (1) career education (1) romance computers phones travel sports fashion health religion celebs tv-movies music-radio literature webmasters programming techmarket Links: (1) (2) (3) (4) (5) (6) (7) (8) (9) (10) Nairaland - Copyright © 2005 - 2024 Oluwaseun Osewa. All rights reserved. See How To Advertise. 38 |