Indentation here is rather odd. Indent one level for any multi-line HTML elements, and one level within Ruby blocks as you would in straight Ruby code. Example:

```
<p>
  Is a favorite?:
  <% if @bookmark.is_favorite %>
    <img src="http://sweetclipart.com/multisite/sweetclipart/files/check_mark_green.png">
  <% else %>
    <img src="http://upload.wikimedia.org/wikipedia/en/thumb/b/ba/Red_x.svg/600px-Red_x.svg.png">
    </li>
  <% end %>
</p>
```

With this better indentation, we can actually spot a problem with the markup: There is a stray `</li>` closing tag inside the second branch of the `if`.