Hello since today, the Dark Theme Discourse logo is giant and hides the toolbar:
No problem in Light Theme (the default theme?).
Same on mobile where it pushes the right part of the header slightly out of screen:
yes - just noticed that - wasn’t happening earlier so must be a very recent change
same happening in Grey Amber
3 Likes
@Johani, do you think it’s related to this?
https://meta.discourse.org/t/smart-mobile-header/99846/73?u=jesus2099
Actually that Smart mobile header does not have this bug.
This giant logo only shows up when the Smart header (topic title and mini logo, etc.) is not showing, that is, when we are at the top of the page.
2 Likes
Oh, good, it just has been fixed. 
What do you mean? Logo is still huge here on Meta as I see it…
Yes, the “alternative logo” component we were using for the dark themes isn’t compatible with the latest improvements to core. I’ve added a temporary fix to meta, made some extensibility improvements to core and made a pull request to the component so that it should be more future-proof.
https://github.com/hnb-ku/discourse-alt-logo/pull/1
@danekhollas try refreshing your browser, it should be ok now 
12 Likes
im using hosted-discourse and just got our site running, when I use a bad url, ie qa.fmod.com/asdf - the home icon becomes gigantic. I see the normal sized image one has
img[Attributes Style] {
height: 36px;
}
in the stylesheet but that is missing when visiting a bad page. I’ve forced it into the stylesheet right now to fix it but would like to remove this hack.
edit: You should be able to see the forced height value I put in , with the link above. Unticking that in the browser inspector will show the bad behaviour.
5 Likes
Merged it and updated here on Meta, much appreciated 
Should be fixed via
https://github.com/discourse/discourse/commit/e2214b50f34c22d44645c641968acb89f81287dc
Thanks for reporting the issue @bpaterson2000 
5 Likes
FYI this was happening to me but when I removed the duplication of the smart mobile header theme (since committed to core I understand) it was fixed (and before this commit)
1 Like
This topic was automatically closed after 2 days. New replies are no longer allowed.