You have a single `li` close tag outside of your `each` loop here, and an `li` open tag inside the loop that will be output once for each bookmark -- so this will generate invalid HTML. A good way to keep this straight is to always add the close tag (or `end` for a Ruby block) at the same time as you open the tag/block.