Unfortunately, I cannot see changes ...
The logo is a 75x75px PNG image.

this cannot be tested with com_patchtester

you can use the Prebuilt packages

I don't know how to test patches with prebuild packages.

Click show all checks -> Click details next to "Download — Prebuilt packages are available for download."

Then download one of the zip files - and install it just like you would a fresh install of Joomla.

Thanks @PhilETaylor

I have downloaded and installed the "Joomla_4.0.2-dev+pr.35337-Development-Full_Package" package on my localhost.
I can confirm that the problem with the size of the logo is now solved !
Thanks

@mariantanase Please go to https://issues.joomla.org/tracker/joomla-cms/35337

Login on the top right with your github account
Next click on "Test this" and check "Tested successfully" to mark the PR as tested

I have tested this item ✅ successfully on fdc5a13

I have tested this item ✅ successfully on fdc5a13

RTC

@mariantanase It's now up to the release leads if they will merge this PR. Changes are likely that they will do this, but we have to wait till there is a new release. Hope you don't mind waiting. @bembelimen please consider merging ?

Can we not use rem on the width/height of the svg? fixing the width on the svg itself kinda defeats the purpose of it being an SVG.

As far as I know SVG doesn't support rem units as width and height attribute. Setting a width and a height in the SVG doesn't have impact on the scalability. The width ad height attributes in the SVG are just a default size, If no size is given, the SVG will always scale 100% of it's container width. With these attributes set, it will firstly respect these values. With CSS the SVG can be scaled up or down if needed.

Remove the width defined in the CSS, and use em (not rem) as a unit of measurement inside the SVG itself.

@C-Lodder I converted the px to em units which is a good option I think.

Just a couple of things to take into consideration with this change:

1. When using alternative image formats (PNG, JPG, etc), the user will need to ensure tthe image is the correct size when uploading. If you upload a 800x200px image, the template's CSS will NOT resize it for you.
2. When using SVGs, the width will need to be set inside the SVG file priors to uploading.

Agree... a bit common sense is needed if you build a site ;)
We could set a max width if that's preferred? Setting a large (full width) banner image can be nice on some cases...

Banner image will respect the max width of 100%

You are DOING IT WRONG. Width and height are unitless: https://developer.mozilla.org/en-US/docs/Web/SVG/Tutorial/Positions

Check https://www.youtube.com/watch?v=YM3KszYmn58 and/or https://speakerdeck.com/jensimmons/using-the-html-width-and-height-attributes-to-improve-web-page-loading

@dgrammatiko You are right! Thanks for checking! What would be the best option? width and height attributes with no units (will be used as px)?

What would be the best option? width and height attributes with no units (will be used as px)?

The width and height attributes are used by the browser to calculate the needed space (the aspect ratio basically) that the image will occupy once loaded. Defining them means that the browser will not jump when the image is downloaded and rendered. The styling is always done in the CSS, (eg img {width: 10em, height:auto}, or whatever). In short, the width and height attributes have nothing to do with the sizing of the displayed image. It is really worth spending the time and checking the links I posted above. There's a huge misconception in this topic (I also used to believe the wrong thing for way too long)

Can someone retest this please

your wish is my command

I have tested this item ✅ successfully on 1746574

I have tested this item ✅ successfully on 1746574

@dgrammatiko is this right by you now?

It’s fine

Thanks