Fix logo size

[boherm]

Questions	Answers
Description?	After tests review for #600 (review), we need to fix the logo size when the uploaded image is taller than longer.
Type?	bug fix
BC breaks?	no
Deprecations?	no
Fixed ticket?	Not yet.
Sponsor company	PrestaShop SA
How to test?	See @florine2623 review in #600 (review)

--

Additional info issue from the release tests

• Related to Unusable menu dropdown when store logo is tall #546
  Tested with this file :

Here's the result ❌

Screen.Recording.2024-02-23.at.15.02.30.mov

[tblivet]

Looks good to me 👍
If we limit the height, maybe we also need to do the same for the width ?
Like max-width: 150px; ?

[SharakPL]

Like max-width: 150px; ?

Yes, but from my experience it should be around 250px

[AureRita]

Hi @boherm

Thank you for your PR, currently I tested it and, yes, the logo is a little bit smaller but I think it's not enought as you can see :

FYI : you can scroll under the logo

Can you correct it ?

Waiting for your feedback

[boherm]

Hi @AureRita, thanks for the review!

This is only on the homepage or on a specific page?

It looks like you haven't properly rebuild hummingbird theme or don't clear the browser cache. 🤔
Have you done this steps?

@SharakPL, @tblivet,
So, do you prefer adding max-width: 250px; property and also removing .img-fluid (that is totally bypassed by this rule)?

[AureRita]

Hi @boherm

Thank you for your PR, I tested it and it seems to works as you can see :

recording.140.webm

Tested on develop and 8.1.x

Because the PR seems to works as expected, It's QA ✔️

Thank you