Veuse said
iKreativ said
1280×800.No way. That was more like 800px width.
Ohh, my bad, yeah switch them numbers around lol typo!
What res. is that with the menu-screenshot? Somewhere between 1024 and 768? Watch it on i.e. an ipad, and it looks fine.
1280×800.
I had not styled the collapsed menu for this skin. It was there, but with white text on white background.
Fair enough.
Btw; To generic for what? As far as I can tell, some of the biggest top-sellers here are super-generic.
Y’know, just doesn’t seem to grab attention, lack of ‘flare’...
Too generic.
Media queries need work: Nav looks a mess: http://d.pr/i/o4Ba and below that res there is no nav…
Spacing at this res: http://d.pr/i/JDlL
RubenBristian said
iKreativ said
CodeBotics saidDoesn’t mean it’s right
Also, HTML5 does allow you to wrap block-level elements with a tags.![]()
Why not?
If HTML5 allows me to do something, it means it is right. This is called “improvement” or “evolution”. Developers always wanted the a to be a block element. Now it is, and it should be used as a block element!This is the most funny sentence that i heard all day
Yes, if used in a relevant context. Example:
This would seem appropriate.
<article>
<a href="full_story/">
<h3>Some article title.</h3>
<p><img src="article.jpg" alt="Image for article." />Etiam scelerisque, nunc ac egestas consequat, odio nibh euismod nulla, eget auctor orci nibh vel nisi.</p>
</a>
</article>
This doesn’t. It’s not semantic or relevant. Just looks plain stupid. I’m positive all the divs don’t need links, just the img inside the li.
<a href="someplace/">
<div class="something">
<div class="something-else">
<ul class="even-deeper">
<li><img src="article.jpg" alt="Image for article." /></li>
</ul>
</div>
</div>
</a>
That’s like saying “hey, we’re allowed to use a as block-level element, lets wrap everything inbetween the body tags in an a. Just plain stupid.
-Scott
CSS bug?
Brutally honest, it’s terrible.
Go back to the drawing board, learn basic design principals, study spacing, hierarchy, rhythm, color theory then try again.
-Scott
Sure they do, everyone is welcome!
CodeBotics said
Also, HTML5 does allow you to wrap block-level elements with a tags.
Doesn’t mean it’s right 
Ok, here goes, it’s pretty harsh but I think that helps improvement!
DESIGN :
I think the screenie says it all. http://awesomescreenshot.com/074lvuxf8CODE :
You could really improve the performance. Try combining the JS files and CSS files, less HTTP requests and put all that inline JS into the JS files.
Nesting. h1 then a.
Broke.
Well, there you go, hope it helps!
-Scott

No way. That was more like 800px width.
This is the most funny sentence that i heard all day