Code Review

Also, using a generic 'ul' tag here seems ... too generic.  I'd give it a class (errors?) just for clarity and to lessen my personal anxiety.