More or less in decreasing order of importance:
- [
rel
is misused][1]. Instead of<span rel="x">
it should be<a href="#x">
; use<a href="#">
for the undecided ones. You could start using [thehashchange
event][8] too to make things a bit nicer. - [
<aside>
is not used semantically][2]. - Put
"use strict";
at the top of your JavaScript. - That would give you errors if you did something silly, like, say, forgetting a
var
and thus creating a global variabletarg
orscrollTo
. preventDouble
is not used.- In fact, your code could use some JSHinting for both [correctness][6] and [style][7].
- The
section#decisionList
seems to be silly wrapper used only for styling; get rid of it. - JavaScript at the bottom of the page (before
</body>
) for best performance. This would also get rid of the need for the$(document).ready
wrapper, but of course you'd still want to use an [IIFE][5] to avoid polluting the global scope.