Got a new'n to review www.indigenousdesigns.com
Moderator: General Moderators
Got a new'n to review www.indigenousdesigns.com
We just wrapped up www.indigenousdesigns.com. There will be a store to follow. For now, please nitpick and let me know every little detail you don't or do like about it.
-
nickvd
- DevNet Resident
- Posts: 1027
- Joined: Thu Mar 10, 2005 5:27 pm
- Location: Southern Ontario
- Contact:
The first thing that I noticed was the sub navigation not appearing when javascript is turned off. Then I noticed on each main page a duplication of those subnav items, meaning that I can still get to those deeper pages even though the main navigation "widget" is "broken"...
That gets one big THUMBS UP from me...
That gets one big THUMBS UP from me...
- RobertGonzalez
- Site Administrator
- Posts: 14293
- Joined: Tue Sep 09, 2003 6:04 pm
- Location: Fremont, CA, USA
It was actually surprisingly easy to make, Everah! Of course, I used jQuery because I am crippled without it, LOL. It could use a little clean-up, because admittedly it took some trial and error to get it working properly, but here's the code so you don't have to try and extract it yourself:
Code: Select all
//<![CDATA[
function findPos(obj) {
var curleft = curtop = 0;
if (obj.offsetParent) {
curleft = obj.offsetLeft
curtop = obj.offsetTop
while (obj = obj.offsetParent) {
curleft += obj.offsetLeft
curtop += obj.offsetTop
}
}
return [curleft,curtop];
}
$(function(){
$('#menu li.outer').hover(
function() {
pos = findPos(this.childNodes[0]);
$(this).find('ul.flyout').css('left', pos[0]).show();
},
function() {
$(this).find('ul.flyout').hide();
}
);
})
//]]>
Code: Select all
.flyout {
display: none;
position: absolute;
top: 73px;
height: 30px;
list-style-type: none;
}
.flyout li {
display: inline !important;
padding: 0 !important;
}
.flyout li a {
padding-right: 1.25em;
color: #000 !important;
}Code: Select all
<ul id="main-menu">
<li class="outer"><a href="http://www.indigenousdesigns.com/" class="first" id="main-link-home"><span>Home</span></a></li>
<li class="outer"><a href="company/" id="main-link-company"><span>Company</span></a>
<ul class="flyout">
<li><a href="company/index.php"><span>Our Story</span></a></li>
<li><a href="company/philosophy.php"><span>Philosophy</span></a></li>
<li><a href="company/invest.php"><span>Invest in us</span></a></li>
<li><a href="company/contact.php"><span>Contact Us</span></a></li>
<li class="last"><a href="company/sitemap.php"><span>Site map</span></a></li>
</ul>
</li>
<li class="outer"><a href="garment/" id="main-link-garment"><span>Garment Info</span></a>
<ul class="flyout">
<li class="first"><a href="garment/index.php"><span>Fibers</span></a></li>
<li><a href="garment/handmade.php"><span>Handmade</span></a></li>
<li><a href="garment/artisans.php"><span>Artisans</span></a></li>
<!--<li class="last"><a href="garment/product-care.php"><span>Product Care</span></a></li>-->
</ul>
</li>
<li class="outer"><a href="shop/" id="main-link-shop"><span>Shop</span></a>
<ul class="flyout">
<li class="last"><a href="shop/finder.php"><span>Find a store</span></a></li>
</ul>
</li>
<li class="outer"><a href="wholesale/" id="main-link-wholesale"><span>Wholesale</span></a></li>
<li class="outer"><a href="news/" class="last" id="main-link-news"><span>News</span></a></li>
</ul>
<!--/main-menu-->- RobertGonzalez
- Site Administrator
- Posts: 14293
- Joined: Tue Sep 09, 2003 6:04 pm
- Location: Fremont, CA, USA
- superdezign
- DevNet Master
- Posts: 4135
- Joined: Sat Jan 20, 2007 11:06 pm
+
- The colour scheme
- The top menu functionality - it's simple, straightforward, unobtrusive & easy to navigate
- The site layout & theme & everything really lend themselves well to the company & the type of of business it is - it fits right in
- The fact that the content isn't centered bothers me - it has no effect on the navigability or any other measurable metric of site navigation, just a personal thing.
- The appearance of the navigation: I don't like the way bolded Verdana looks. Also, the buttons have a 3D look, but everything else in the site is flat. I'd suggest getting rid of the bevel & change the font to the same size & style as the nav on the bottom.
- The navigation on the sub pages would look better if it was in a bit - not flush to the side of the window. There's too much whitespace between the side navigation & the page content.
- Have the border around the header words & image wrap around the photo too.
- I'm not really liking the rounded navigation title on the subpages - maybe it's too short, or maybe (read: more than likely) I'm just being picky.
Real programmers don't comment their code. If it was hard to write, it should be hard to understand.
LOL everything on your negative list has been addressed... and were all insisted on by the client (the reason I chuckle is because that was like MY EXACT LIST of things I didn't like about it, but the client insisted on)
EDIT: Oh... and thanks for the kind words as well as the negative. You guys are great.
EDIT AGAIN: Oh and I can't take credit for the design... it was actually designed by somebody else.
EDIT: Oh... and thanks for the kind words as well as the negative. You guys are great.
EDIT AGAIN: Oh and I can't take credit for the design... it was actually designed by somebody else.