Fix theme logo position and shape

[jospoortvliet]

Fix from #344 (comment)

• fix logo in top bar
• fix logo on login
  Working:
• logo's that are higher than wide:
  
  
• square-ish ones:
  
  

Still broken:

• upon first apply, the logo is stretched. Can't figure out why, a reload fixes it and makes it look good.
  
• logo's which are wider are top-aligned, can't get them to vertically center:
  

I can't figure out how to fix those. Leave it up to somebody with actual CSS fu.

[mention-bot]

@jospoortvliet, thanks for your PR! By analyzing the annotation information on this pull request, we identified @LukasReschke, @schiessle and @juliushaertl to be potential reviewers

[MorrisJobke]

cc @jancborchardt

[LukasReschke]

Awesome. I'll hijack to fix the unit tests, @jospoortvliet :-)

[LukasReschke]

Tests adjusted at 8457c61, now I'll leave that to the CSS gurus 😉

[juliusknorr]

I'll take over to fix the remaining css issues.

[juliusknorr]

Should be done:

• Fixed preview alignment
• wider logos are now centered

I guess this is now ready for review.

[go2sh]

Backport would be nice.

[MorrisJobke]

Tested and works 👍

[MorrisJobke]

@rullzer @Mar1u5 @nickvergessen @schiessle Review would be nice :)

[schiessle]

looks good 👍

[MorrisJobke]

Backport would be nice.

cc @karlitschek

[karlitschek]

cool. please backport 👍

[juliusknorr]

I'll create a backport for this, just for Nextcloud 10 or also for Nextcloud 9 @karlitschek @MorrisJobke

[karlitschek]

i think both would be good

[jancborchardt]

Awesome work @juliushaertl! :)