This code could be condensed a bit. The `== true` is not needed, and the `else bookmark.favorite` does nothing because there is no `=` for page output. The `</dt>` closing tag is also only output if the bookmark is not a favorite -- so if it is a favorite, this will result in invalid HTML. (actually, this is invalid HTML anyway because definition lists must have pairs of `dt` and `dd`, but this is all `dt`)