MakD / zombie-release

A theme for Jellyfin 10.9.x combining the best from the Public Themes
46 stars 4 forks source link

mobile-alt-layout breaking logo in desktop app #16

Open stpnwf opened 2 weeks ago

stpnwf commented 2 weeks ago

Hey, I was looking at the changes you've made to Zombie to see if there were cool ones I could backport to Zesty, when I found this bug. It only happens in the Desktop app and in Firefox. Issue doesn't happen in Chrome/Chromium. And it is triggered by mobile-alt-layout.css. Great job on the changes btw. Your CSS is so clean hahaha

Screenshot from 2024-09-06 09-44-22

stpnwf commented 1 week ago

Using display: none; and display: block; is what is causing it. Either one of these solved it for me.

#itemDetailPage .itemName.infoText.parentNameLast > bdi:first-child,
#itemDetailPage .parentName > bdi:first-child {
  opacity: 0;
}
.hide + .detailPageWrapperContainer .itemName > bdi:first-child,
.hide + .parentName > bdi:first-child {
  opacity: 1 !important;
}

or

#itemDetailPage .itemName.infoText.parentNameLast > bdi:first-child,
#itemDetailPage .parentName > bdi:first-child {
  visibility: hidden;
}
.hide + .detailPageWrapperContainer .itemName > bdi:first-child,
.hide + .parentName > bdi:first-child {
  visibility: visible !important;
}

I couldn't figure out how to get the title to disappear and have it show up when there was no logo, so I ended up having it being shown all the time in Zesty... Well, no more. Thanks to this snippet. Now I just gotta figure out how to make the logo a link.