More or less in decreasing order of importance:
- [
relis misused][1]. Instead of<span rel="x">it should be<a href="#x">; use<a href="#">for the undecided ones. You could start using [thehashchangeevent][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
varand thus creating a global variabletargorscrollTo. preventDoubleis not used.- In fact, your code could use some JSHinting for both [correctness][6] and [style][7].
- The
section#decisionListseems 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).readywrapper, but of course you'd still want to use an [IIFE][5] to avoid polluting the global scope.