Enable support for ultra-wide screens #315
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This was referenced Nov 21, 2024
thornbill
approved these changes
Dec 14, 2024
|
I had a similar problem with my tizen app. Displaying in a 16:9 box on ultrawides. 32:9 in my case. What helped fix the problem was adding: This should fix the app on ultrawides. |
According to this,
I don't think we need to add it to the main index.html (it just redirects) - it's already there in jellyfin-web. |
The samsung documentation says it should "work" but it just doesn't. I've tried it. "extensive" had no effect on how the app was displayed. It still was in a 16:9 box with black bars all around it when playing ultrawide videos. Only when changing to "fullscreen" (which I saw some other tizen apps were using) the app took the whole width of the screen and properly displayed 32:9 videos. I think "fullscreen" should work for both 21:9 and 32:9 displays as it seemingly instructs to take up the whole available width. The display model I was testing on was SH37C. And for my case it worked perfectly.
In that case yes it should be fine. I was just saying what changes I had to make. Of course it would be best to test on a proper 21:9 display but I think "fullscreen" should fix it. |
|
I tried only changing the config.xml file and installed it on my monitory, but it didn't seem to work. |
|
I tried rewriting all steps to work in wsl storage only (without /mnt/c/...) and I got some error after changing config.xml and typing in the command for npm jellyfin-tizen (last step of npm) Somehow I am getting this error after: |
|
@CreaGab from your logs: The second |
Thanks for noticing! Unfortunatly adding one point didn't solve the issue. I had to add the environments like this: export USE_SYSTEM_FONTS="1" and export JELLYFIN_WEB_DIR="../jellyfin-web/dist". Then the command worked. Unfortunatly it still won't display fullscreen even with the file changed like you said. New command history: |
dmitrylyzo
commented
Dec 20, 2024
|
Just for your info, I won't be able to provide updates until 7th of January. I wish you all merry Christmas and a good start in 2025! |
|
Update: |
|
This at least works in the Tizen 8 emulator. But we probably need to limit the width of the "site", because now it is getting too wide and the item cards are getting too big. UPD: |
'extensive' enables 21:9 'fullscreen' enables 32:9
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
According to https://developer.samsung.com/smarttv/develop/guides/base_screen_resolution.html
Changes
Enable support for ultra-wide screens.
Issues
Fixes #231