Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix backdrop being used on large screens in mobile layout #3618

Merged
merged 1 commit into from
May 5, 2022

Conversation

thornbill
Copy link
Member

@thornbill thornbill commented May 5, 2022

Changes
This fixes an issue where the backdrop was enabled on the item details screen on large devices (tablets) that use the mobile layout. The backdrops are not compatible with the header styling used for the mobile layout.

Issues
As you can see in the following screenshot, the header image and backdrop are not aligned. The header image also scrolls with the page while the backdrop remains fixed in the mobile layout.

4B35AF34-44F1-4C56-9D91-97A3B241F52D

@thornbill thornbill added bug Something isn't working stable backport Backport into the next stable release labels May 5, 2022
@sonarcloud
Copy link

sonarcloud bot commented May 5, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@thornbill thornbill merged commit f2c27dc into jellyfin:release-10.8.z May 5, 2022
@thornbill thornbill deleted the fix-backdrop-mobile branch May 5, 2022 20:01
jellyfin-bot pushed a commit that referenced this pull request May 21, 2022
Fix backdrop being used on large screens in mobile layout

(cherry picked from commit f2c27dc)
Signed-off-by: Joshua Boniface <joshua@boniface.me>
@jellyfin-bot jellyfin-bot removed the stable backport Backport into the next stable release label May 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants