SITE REVIEW: OhDesignX

  • righteous_trespasser
  • Scuffle
  • Genius
  • User avatar
  • Posts: 6230
  • Loc: South-Africa

Post 3+ Months Ago

http://www.ohdesignx.com ...
  • Anonymous
  • Bot
  • No Avatar
  • Posts: ?
  • Loc: Ozzuland
  • Status: Online

Post 3+ Months Ago

  • RedBMedia
  • Proficient
  • Proficient
  • User avatar
  • Posts: 315

Post 3+ Months Ago

I hate it!.....LOL.....no seriously, i like it....i like the simplicity of it. However, i don't think white text on gray isn't very appealing to the eye, and its lack of color tends to be boring....i would like to see more color and maybe a textured background...I like the layout and the simple structure.
  • digitalMedia
  • a.k.a. dM
  • Genius
  • User avatar
  • Posts: 5149
  • Loc: SC-USA

Post 3+ Months Ago

I like your color combinations and general content organization.

On the critical side, I am very uncomfortable with the paragraphs being centered, especially when they're stretched out so far. It takes more effort to read when your eye has to search for the beginning of the next line.

Under the hood, you might consider using a UL for your menu. That's generally considered better for accessibility. I'm not crazy about your use of non-breaking spaces. Also, for good form, you might consider using multiple P tags, instead of using two BRs.

I'd like to see you move more content to the front page, as well. I think that would add more interest to your front page and give the user the feeling that there is more to explore. Perhaps you could randomly place one of your portfolio sites and a brief blurb about the client's needs and the solution you provided for them. Maybe you could add a special pricing offer.

I hope that didn't sound too critical. Overall, great job!
  • celandine
  • Mastermind
  • Mastermind
  • User avatar
  • Posts: 2008
  • Loc: Belgrade, Serbia

Post 3+ Months Ago

it looks lovely, clean and simple. I would justify the text. the front page needs to be longer - I'd like to see it at least the length of the portfolio page.... I like the color scheme, both the gray/white and the blue/green logo that gives some color focus. Though I might try to give the logo some more definition/ contrast/ saturation. it might be just the monitor I'm currently on but it looks a tiny bit washed out..............

otherwise I love it
  • jameson5555
  • Bronze Robot
  • Bronze Member
  • User avatar
  • Posts: 575
  • Loc: Phoenix, AZ

Post 3+ Months Ago

Lookin' good! All the comments so far are right on. I agree that centered paragraphs generally aren't good.

Also, on the front page, it seems like that first paragraph should be broken up. Maybe even into some fancy custom bullet points or something... :)

Another thing I noticed is that on the pricing page, the first package includes 5 business cards? That seems kinda strange to me.. does that mean 5 business card designs or 5 actual cards?
  • righteous_trespasser
  • Scuffle
  • Genius
  • User avatar
  • Posts: 6230
  • Loc: South-Africa

Post 3+ Months Ago

digitalMedia wrote:
Under the hood, you might consider using a UL for your menu. That's generally considered better for accessibility. I'm not crazy about your use of non-breaking spaces. Also, for good form, you might consider using multiple P tags, instead of using two BRs.

Okay, I can do that, no problem ... Can you tell me what the reason for that is? it seems irrelevant to me ...

digitalMedia wrote:
I'd like to see you move more content to the front page, as well. I think that would add more interest to your front page and give the user the feeling that there is more to explore. Perhaps you could randomly place one of your portfolio sites and a brief blurb about the client's needs and the solution you provided for them. Maybe you could add a special pricing offer.

That's brilliant, I never thought about that ...Why, I wouldn't know ...

digitalMedia wrote:
I hope that didn't sound too critical. Overall, great job!
One can never be too critical ... And thanks!

celandine wrote:
Though I might try to give the logo some more definition/ contrast/ saturation. it might be just the monitor I'm currently on but it looks a tiny bit washed out
Yeah, it does a bit ... will only be able to fix that when I'm at home tonight ... (Only 6:30AM now)

jameson5555 wrote:
Another thing I noticed is that on the pricing page, the first package includes 5 business cards? That seems kinda strange to me.. does that mean 5 business card designs or 5 actual cards?

It's 5 business card designs ... I'll try and make it clearer ...

Then ... If you all were wondering why I added nothing but a link to my first post, it was just to get "untampered opinions" if you know what I mean ...
  • JasonStanley
  • Graduate
  • Graduate
  • JasonStanley
  • Posts: 124
  • Loc: London, England

Post 3+ Months Ago

I think there is a distinct lack of color in the design. The problem I see is the background gray is too light to use white text with it. Thus the text is slightly difficult to read. It would be interesting to hear how someone with a minor visual impairment gets along with the site.

The quality of the header could be improved. Perhaps a logo could be added. The navigation could be spruced up and the whole site could become a little a less gray.

As far as SEO goes you could definitely improve your page titles. Meta tags wouldn't go amiss either.

My main issue is that you have called yourself ohdesign... wheres the 'oh' in the design? I can see that you were going through simplicity in design however I think this site is far too close to bland.
  • righteous_trespasser
  • Scuffle
  • Genius
  • User avatar
  • Posts: 6230
  • Loc: South-Africa

Post 3+ Months Ago

JasonStanley wrote:
I think there is a distinct lack of color in the design. The problem I see is the background gray is too light to use white text with it. Thus the text is slightly difficult to read. It would be interesting to hear how someone with a minor visual impairment gets along with the site.
Okay so this seems to be an issue, I'll fix this to make tha background darker, but that would probably only happen over the weekend.
JasonStanley wrote:
The quality of the header could be improved.
I have noted that and will address it aswell with the page colour
JasonStanley wrote:
As far as SEO goes you could definitely improve your page titles. Meta tags wouldn't go amiss either.
Thanks for that, it's something I tend to miss frequently ...
  • digitalMedia
  • a.k.a. dM
  • Genius
  • User avatar
  • Posts: 5149
  • Loc: SC-USA

Post 3+ Months Ago

Quote:
Okay, I can do that, no problem ... Can you tell me what the reason for that is? it seems irrelevant to me ...


For accesability, it has to do with how non-visual browsers read a page. Plus, the idea of XHTML is that it is essentially XML and therefore very portable to a great number of applications, not just web pages. Putting a menu in a UL separates a menu from textual content, logically..

Adding non-breaking spaces is a matter of formatting. We use CSS to seperate content and form...that's the theory anyway. The same applies to creating multiple paragraphs in one P tag. As XML sees it, you've only got one paragraph.

IMO, the best practice is to view the markup of any page without any styles. Does it make sense? HTML was originally intended to reproduce a printed document as hyper text. Everything we've done to HTML since IE3/NS3 is a sort of bastardization of that idea. Still, we can develop documents correctly and provide styling for them in a seperate layer(CSS) for web presentation.

I hope all that made sense. It's early morning where I am, I'm only on my second cup of coffee and I'm hurrying so I can take the kids to school.
  • righteous_trespasser
  • Scuffle
  • Genius
  • User avatar
  • Posts: 6230
  • Loc: South-Africa

Post 3+ Months Ago

cool, it does make sense ... I'll do all this some time this weekend ... thanks for the explanation ...
  • zhaira
  • Graduate
  • Graduate
  • zhaira
  • Posts: 223

Post 3+ Months Ago

I like your site but you need some improvements on the header , logo and navigation menus. Functionality and usability of the site is important for the site. Content is another factor on which the users usually read the site.
  • Bogey
  • Genius
  • Genius
  • Bogey
  • Posts: 8388
  • Loc: USA

Post 3+ Months Ago

The page is valid, although I didn't expect anything less than that assuming that you coded the site :lol:

I hate those stuff you do on contact... just giving the email with a mailto:... I prefer on-line forms more simply because those mailto:'s don't work for my... how come? Everytime I send something like that I receive the messages I send in the Outlook... so everytime I end up either copying/pasting the email address to my email and sending through that way, copying/pasting the email address from the source or not being able to contact at all... just my personal opinion...

Otherwise, nice layout, good color scheme, awesome links...
  • righteous_trespasser
  • Scuffle
  • Genius
  • User avatar
  • Posts: 6230
  • Loc: South-Africa

Post 3+ Months Ago

Okay cool, I'll try and teach myself how to write a mailer script (PHP) or I'll try and find one somewhere ...

And yes, @zhaira ... I know the header needs work, I just put it together in paint quickly because it was in two loose pieces, and then paint didn't save high enough quality ...

zhaira wrote:
Functionality and usability of the site is important for the site.

So what does this mean exactly, do you find something "not useable/functional" enough ... please explain ...

zhaira wrote:
Content is another factor on which the users usually read the site.

I would hope so ... Are you saying that my content isn't good enough, or is this just some random sentence ...
  • spork
  • Brewmaster
  • Silver Member
  • User avatar
  • Posts: 6251
  • Loc: Seattle, WA

Post 3+ Months Ago

Ahh R_T sorry man, haven't been in this forum for a while and didn't even see your thread, otherwise my review would have been here much sooner.

Overall, a very good first impression when I opened the site initially. I'm not so sure I'm crazy about each page title being CAPITALIZED. Can't put my finger on it; it just seems... odd.

I agree with the whole nbsp/<ul>/<br> comments above. Don't really have much to add in that respect.

The color scheme works well, is pleasing to the eye (or at least to my eyes), and the text is readable.

One little gripe -- when I am browsing a site, I tend to expect the main top logo to act as a hyperlink to the homepage. Yours does not. I cried a little. Not a big deal though, just figured it would be something to consider.

Nice job.
  • Bogey
  • Genius
  • Genius
  • Bogey
  • Posts: 8388
  • Loc: USA

Post 3+ Months Ago

righteous_trespasser wrote:
Okay cool, I'll try and teach myself how to write a mailer script (PHP) or I'll try and find one somewhere ...


Here's one I created some time ago :D (Needs SMTP)

Code: [ Select ]
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
  "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" lang="en" xml:lang="en">
<head>
<title>Form Test</title>
<style type="text/css">
body {
 background-color: #CECECE;
}
fieldset#form {
 background-color: #999999;
}
</style>
</head>
<body>
<?php
//form validation
//setting the values
 $name = $_POST['name'];
 $email = $_POST['email'];
 $who = $_POST['who'];
 $subject = $_POST['subject'];
 $message = $_POST['message'];
//subject validation
 $show_form = true;
 if(!empty($_POST['submit']))
 {
  if(empty($name)) {
  $error = "<li>You need a name</li>";
  $s = false;
  $n = false;
  } else {
  $success = "<b>Name:</b> <u>$name</u><br />";
  $s = true;
  }
  if (empty($email)) {
  $error .= "<li>You need an email</li>";
  $s = false;
  $e = false;
  } elseif (!preg_match("/^[A-Z0-9._%-]+@[A-Z0-9][A-Z0-9.-]{0,61}[A-Z0-9]\.[A-Z]{2,6}$/i", $email)) {
  $error .= "<li>You need a valid E-Mail</li>";
  $s = false;
  $e = false;
  } else {
  $success .= "<b>E-mail:</b> <u>$email</u><br />";
  $s = true;
  }
 //email validation
 //subject validation
  if (empty($subject)) {
  $error .= "<li>You need a subject</li>";
  $s = false;
  $su = false;
  } else {
  $success .= "<b>Subject:</b> <u>$subject</u><br />";
  $s = true;
  }
 //who validation
  $to = $who;
 //message validation
  if (empty($message)) {
  $error .= "<li>You need a message</li>";
  $s = false;
  $m = false;
  } else {
  $success .= "<b>Message:</b> <u>$message</u><br />";
  $s = true;
  }
 }
?>
<?php
 if($show_form === true) {
?>
<p>Please fill in the form below...</p>
<form action="testingform.php" method="post">
<fieldset id="form">
<?php if($n === false) { echo '*'; } ?>Name:<br />
<input type="text" name="name" /><br />
<?php if($e === false) { echo '*'; } ?>E-mail:<br />
<input type="text" name="email" /><br />
To whom are you sending this e-mail to?<br />
<select name="who">
<!-- You can add more emails without changing the PHP //-->
<option value="email1@domain.com">Option 1</option>
<option value="email2@domain.com">Option 2</option>
</select><br />
<?php if($su === false) { echo '*'; } ?>Subject:<br />
<input type="text" name="subject" /><br />
<?php if($m === false) { echo '*'; } ?>Message:<br />
<textarea name="message" rows="10" cols="25"></textarea><br />
<input type="submit" name="submit" value="submit" /> <input type="reset" value="reset" />
</fieldset>
</form>
<p><small>All of them are required</small></p>
<?php
 } else {
 echo 'Thanks for your post '. $name;
 echo $success;
 }

//Success or error output
 if(!empty($_POST['submit']))
 {
  if($s === false) {
  echo '<ul>'. $error .'</ul>';
  echo '<p><a href="testingform.php">Redo</a></p>';
  $show_form = false;
  } elseif($s === true) {
  mail($to, $subject, $message, "From: '. $email .'\r\n");
  $show_form = false;
  } else {
  echo 'There was an unknown error in the mailing system.';
  $show_form = false;
  }
 }
?>
</body>
</html>
  1. <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
  2.   "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
  3. <html xmlns="http://www.w3.org/1999/xhtml" lang="en" xml:lang="en">
  4. <head>
  5. <title>Form Test</title>
  6. <style type="text/css">
  7. body {
  8.  background-color: #CECECE;
  9. }
  10. fieldset#form {
  11.  background-color: #999999;
  12. }
  13. </style>
  14. </head>
  15. <body>
  16. <?php
  17. //form validation
  18. //setting the values
  19.  $name = $_POST['name'];
  20.  $email = $_POST['email'];
  21.  $who = $_POST['who'];
  22.  $subject = $_POST['subject'];
  23.  $message = $_POST['message'];
  24. //subject validation
  25.  $show_form = true;
  26.  if(!empty($_POST['submit']))
  27.  {
  28.   if(empty($name)) {
  29.   $error = "<li>You need a name</li>";
  30.   $s = false;
  31.   $n = false;
  32.   } else {
  33.   $success = "<b>Name:</b> <u>$name</u><br />";
  34.   $s = true;
  35.   }
  36.   if (empty($email)) {
  37.   $error .= "<li>You need an email</li>";
  38.   $s = false;
  39.   $e = false;
  40.   } elseif (!preg_match("/^[A-Z0-9._%-]+@[A-Z0-9][A-Z0-9.-]{0,61}[A-Z0-9]\.[A-Z]{2,6}$/i", $email)) {
  41.   $error .= "<li>You need a valid E-Mail</li>";
  42.   $s = false;
  43.   $e = false;
  44.   } else {
  45.   $success .= "<b>E-mail:</b> <u>$email</u><br />";
  46.   $s = true;
  47.   }
  48.  //email validation
  49.  //subject validation
  50.   if (empty($subject)) {
  51.   $error .= "<li>You need a subject</li>";
  52.   $s = false;
  53.   $su = false;
  54.   } else {
  55.   $success .= "<b>Subject:</b> <u>$subject</u><br />";
  56.   $s = true;
  57.   }
  58.  //who validation
  59.   $to = $who;
  60.  //message validation
  61.   if (empty($message)) {
  62.   $error .= "<li>You need a message</li>";
  63.   $s = false;
  64.   $m = false;
  65.   } else {
  66.   $success .= "<b>Message:</b> <u>$message</u><br />";
  67.   $s = true;
  68.   }
  69.  }
  70. ?>
  71. <?php
  72.  if($show_form === true) {
  73. ?>
  74. <p>Please fill in the form below...</p>
  75. <form action="testingform.php" method="post">
  76. <fieldset id="form">
  77. <?php if($n === false) { echo '*'; } ?>Name:<br />
  78. <input type="text" name="name" /><br />
  79. <?php if($e === false) { echo '*'; } ?>E-mail:<br />
  80. <input type="text" name="email" /><br />
  81. To whom are you sending this e-mail to?<br />
  82. <select name="who">
  83. <!-- You can add more emails without changing the PHP //-->
  84. <option value="email1@domain.com">Option 1</option>
  85. <option value="email2@domain.com">Option 2</option>
  86. </select><br />
  87. <?php if($su === false) { echo '*'; } ?>Subject:<br />
  88. <input type="text" name="subject" /><br />
  89. <?php if($m === false) { echo '*'; } ?>Message:<br />
  90. <textarea name="message" rows="10" cols="25"></textarea><br />
  91. <input type="submit" name="submit" value="submit" /> <input type="reset" value="reset" />
  92. </fieldset>
  93. </form>
  94. <p><small>All of them are required</small></p>
  95. <?php
  96.  } else {
  97.  echo 'Thanks for your post '. $name;
  98.  echo $success;
  99.  }
  100. //Success or error output
  101.  if(!empty($_POST['submit']))
  102.  {
  103.   if($s === false) {
  104.   echo '<ul>'. $error .'</ul>';
  105.   echo '<p><a href="testingform.php">Redo</a></p>';
  106.   $show_form = false;
  107.   } elseif($s === true) {
  108.   mail($to, $subject, $message, "From: '. $email .'\r\n");
  109.   $show_form = false;
  110.   } else {
  111.   echo 'There was an unknown error in the mailing system.';
  112.   $show_form = false;
  113.   }
  114.  }
  115. ?>
  116. </body>
  117. </html>


CATCH: You'll need to test it out if it actually sends messages... I couldn't test it out since I don't have an SMTP server...
  • righteous_trespasser
  • Scuffle
  • Genius
  • User avatar
  • Posts: 6230
  • Loc: South-Africa

Post 3+ Months Ago

spork wrote:
The color scheme works well, is pleasing to the eye (or at least to my eyes)

Yeah, well, your eyes are pretty easy to please ... I mean you dated my mom not too long ago ...

@Bogey ... thanks, I'll test it some time in the near future ...
  • spork
  • Brewmaster
  • Silver Member
  • User avatar
  • Posts: 6251
  • Loc: Seattle, WA

Post 3+ Months Ago

To be fair, she has very pleasing features. And she bakes a mean apple pie. I can't complain.

Post Information

  • Total Posts in this topic: 17 posts
  • Moderator: Website Reviewers
  • Users browsing this forum: No registered users and 8 guests
  • You cannot post new topics in this forum
  • You cannot reply to topics in this forum
  • You cannot edit your posts in this forum
  • You cannot delete your posts in this forum
  • You cannot post attachments in this forum
 
 

© 1998-2014. Ozzu® is a registered trademark of Unmelted, LLC.