- Issue created by @danrod
- Merge request !52Issue # 3527133: Applied styling for the svg logo to make it look smaller → (Merged) created by danrod
- 🇨🇦Canada danrod Ottawa
Any luck with testing this @esha_kundu ? I'd like to commit to the 8.x-1.x branch as soon as possible.
- 🇨🇦Canada danrod Ottawa
I tested on a clean installation and it looks ok to me @esha_kundu , I'll close the task but I'll give you the credits just in case you did some testing and forgot to mention it here.
Hello @danrod,
Sorry for the late reply, I've tested the MR!52 in Drupal version 10.4.6. The MR applied cleanly, and the logo is visible and is the correct size. However, I found that you have used a static width for the image, due to which when Use the logo supplied by the theme is disabled and a custom logo is used, the logo renders in a very small size. IMO, we can make use of the below snippet.navbar-header .site-branding img { max-width: 80px; max-height: 56px; }
I'm attaching the SS for reference.
Will be glad you hear from you.
Thanks- 🇨🇦Canada danrod Ottawa
Hi @esha_kundu , I tested your changes and It worked like a charm !:
commiting the changes to a new MR and moving this to "Needs Review" just in case anyone has some suggestions.
- Merge request !53Issue # 3527133 by esha_kundu: Added max-width and max-height to the site logo → (Merged) created by danrod
- 🇨🇦Canada danrod Ottawa
I'll merge it to the 8.x-1.x branch for now, if you see any other issues please let me know @esha_kundu thanks !
Automatically closed - issue fixed for 2 weeks with no activity.