Improve navbar

[tagliala]

Moves gravatar near user email
Removes alt attribute from gravatar (useless md5 encoded email)
Also adds support for collapsed navbar

Before

After

[tagliala]

$ CI_ORM=active_record CI_DB_ADAPTER=sqlite3 rspec ./spec/integration/rails_admin_spec.rb:134
Run options:
  include {:locations=>{"./spec/integration/rails_admin_spec.rb"=>[134]}}
  exclude {:mongoid=>true, :skip_active_record=>true}

Randomized with seed 3788
.

Top 1 slowest examples (0.74006 seconds, 99.9% of total time):
  RailsAdmin secondary navigation does not cause error when email is nil
    0.74006 seconds ./spec/integration/rails_admin_spec.rb:134

Finished in 0.74083 seconds (files took 2.86 seconds to load)
1 example, 0 failures

Randomized with seed 3788

😕

[tagliala]

Devise 3.5.1 is causing the failure

[tagliala]

What a lucky man, master branch also fails 🍀

~/dev/rails_admin on master $ CI_ORM=active_record CI_DB_ADAPTER=sqlite3 rspec ./spec/integration/rails_admin_spec.rb:134
Run options:
  include {:locations=>{"./spec/integration/rails_admin_spec.rb"=>[134]}}
  exclude {:mongoid=>true, :skip_active_record=>true}

Randomized with seed 53656
F

Failures:

  1) RailsAdmin secondary navigation does not cause error when email is nil
     Failure/Error: visit dashboard_path
     ActionView::Template::Error:
       undefined method `devise_scope' for #<User:0x007ff5a3889c30>

[dalpo]

@tagliala Could you change in this PR the css selector .brand in .navbar-brand please? 😄
These are stylesheets for the bootstrap2 markup.

If you prefer I could do a PR after the merge of this one.

Thanks!

[dalpo]

👍

[tagliala]

@dalpo thanks

Completely removed the old css rules. I didn't like that old brown label

[tagliala]

Failures due to #2311

[dalpo]

Thanks to you @tagliala! Great work 😄

[tagliala]

could you please re-run the Travis build?

[luizpicolo]

This bothered me a long time 😄 . Great work 👍

[tagliala]

thanks! honestly the "log out" red label is still bothering me but... one thing at a time :)

[dalpo]

@mshibuya I see that you are planning a 0.7 milestone?
When do you think to merge these PRs?

[dalpo]

@tagliala What do you think about to add the 'img-rounded' (or the 'img-circle') class to the avatar image?

Take a look at http://getbootstrap.com/css/#images-shapes

[tagliala]

I'm not anymore a big fan of circled avatars because they cut out part of the user profile picture.

btw no problems for me if authors decide to go for img-circle, I will edit this PR

[rikkipitt]

It could be nice to go for a more modern/consolidated user drop down. I'll try and find an example, but you've probably seen them before - the ones where you click the username/image and a drop down appears with a profile and logout link. It would solve the red "danger-label" as mentioned above and tidy up the top nav bar... Just a thought!

[rikkipitt]

@tagliala and how about a target="_blank" on the "Home" link too? That might make for a nicer experience.

[tagliala]

@rikkipitt it makes sense, but I would leave this decision to the authors. No problem to edit this PR

[mshibuya]

Merged in, thanks!