Review

The font-awesome-ness makes this a bit hard to comprehend, especially mixed in with the erb if-else logic. You might want to think about spliting it into _not_logged_in_links.html and _logged_in_links.html, so that this template could have just something like.
```
if <% current_user %>
  render '_logged_in_links.html'
<% else %>
  render '_not_logged_in_links.html'
<% end %>
```

It would be easier to read and comprehend its intention. You want if possible to avoid a reading having to stop and think "what's going on here?"