HiveBrain v1.2.0
Get Started
← Back to all entries
patternhtmlModerate

HTML Review of template for site - Is this ok for HTML5?

Submitted by: @import:stackexchange-codereview··
0
Viewed 0 times
thistemplatehtml5forsitereviewhtml

Problem

Writing a new site, has anyone got any comments on the HTML here? Is it all semantically correct? Thanks! This will basically be the template, so I want to make sure it's pretty perfect.

```

Welcome to Scirra.com














$(document).ready(function() {
$('#coin-slider').coinslider({ width: 470, height: 261, spw: 8, sph: 4, delay: 7000, effect: 'straight', hoverPause: true });

$('#twitterFeed').twit('Scirra', {
limit: 5,
label: '',
icon: false,
count: 20
});
});













Main Link
Mainy Link
Main Link
Main Link


Another Sub Link
Sub Link
Sub Link
Sub Link
Sub Link








Scirra software allows you to bring your imagination to life





Export your creations to HTML5 pages





Another description of some image





Something motivational to tell people





Latest from Twitter




Main Heading
Some paragraph goes here. It tells you about the picture. Cool! Have you thought about downloading Construct 2? Well you ca

Solution

If you are going with HTML5, I would suggest a few more things:

  • Lose the type="text/javascript" from the script tags - you don't need them in HTML5



  • Same goes with type="text/css" on link tags



  • Keep your self-closing XML syntax consistent - on some tags you have the closing />, on some you don't (it doesn't matter which way, just make it consistent)



  • Consider placing .js files at the bottom of the document - that way the page can start rendering before it's downloaded them all. Often these are required for post-rendering functions anyway - so do your users a favor and put them at the bottom of the document.

Context

StackExchange Code Review Q#866, answer score: 17

Revisions (0)

No revisions yet.