The standard convention for naming instance variables is `@bookmark` for a single bookmark and `@bookmarks` for any collection of bookmarks (whether it's "all" bookmarks or not). This allows some flexibility without having to rename variables -- for instance, if we later changed the index view to only show favorite bookmarks by default, the name `@all_bookmarks` would no longer be accurate, but `@bookmarks` would still be.

Also, `Bookmark.all.order(:name)` can be shortened to `Bookmark.order(:name)`