Fix long playlist strings breaking viewport - #8275
Merged
FreeTubeBot merged 1 commit intoNov 16, 2025
Merged
Conversation
FreeTubeBot
enabled auto-merge (squash)
November 12, 2025 21:19
github-actions
Bot
added
the
PR: waiting for review
efb4f5ff-1298-471a-8973-3d47447115dc
approved these changes
Nov 13, 2025
efb4f5ff-1298-471a-8973-3d47447115dc
requested review from
a user,
absidue and
kommunarr
November 13, 2025 11:42
kommunarr
approved these changes
Nov 15, 2025
PikachuEXE
approved these changes
Nov 16, 2025
github-actions
Bot
removed
the
PR: waiting for review
Member
|
@ozrendev Please do a follow up pull request that reverts these changes on small screen widths. Limiting to 85% of the window width is alright with large window sizes, but on mobile phones where the screen is already really small, only filling 85% of the screen width with content looks odd and requires users to scroll more. When you are making layout changes to FreeTube always test on desktop and mobile window sizes. |
Contributor
Author
Will do! Sorry I totally didn't think about mobile viewport |
PikachuEXE
added a commit
to PikachuEXE/FreeTube
that referenced
this pull request
Nov 18, 2025
* development: (22 commits) Workaround ready-to-show not firing consistently on wayland (FreeTubeApp#8294) Bump electron from 38.4.0 to 39.2.1 (FreeTubeApp#8303) Migrate the WatchVideoInfo component to the composition API (FreeTubeApp#8295) Translated using Weblate (Hungarian) Translated using Weblate (Norwegian Bokmål) Translated using Weblate (French) Bump eslint-plugin-jsdoc from 61.1.12 to 61.2.1 in the eslint group (FreeTubeApp#8298) Replace broken trending with a local API only equivalent (FreeTubeApp#8289) Fix settings layout issues after the Vue 3 migration (FreeTubeApp#8292) Bump lefthook from 2.0.3 to 2.0.4 (FreeTubeApp#8299) Bump shaka-player from 4.16.8 to 4.16.9 (FreeTubeApp#8301) Bump electron-builder from 26.1.0 to 26.2.0 (FreeTubeApp#8302) Fix playlist video view counts on the local API (FreeTubeApp#8290) change hardcoded color app text logoColor (FreeTubeApp#8288) Translated using Weblate (Russian) Translated using Weblate (German) wrap long playlist strings (FreeTubeApp#8275) Translated using Weblate (Bulgarian) Bump js-yaml from 4.1.0 to 4.1.1 (FreeTubeApp#8284) bump workflows node-version to 24 (FreeTubeApp#8279) ... # Conflicts: # src/renderer/helpers/api/invidious.js
PikachuEXE
added a commit
to PikachuEXE/FreeTube
that referenced
this pull request
Nov 18, 2025
* development: Fix playlist video view counts on the local API (FreeTubeApp#8290) change hardcoded color app text logoColor (FreeTubeApp#8288) Translated using Weblate (Russian) Translated using Weblate (German) wrap long playlist strings (FreeTubeApp#8275) Translated using Weblate (Bulgarian) Bump js-yaml from 4.1.0 to 4.1.1 (FreeTubeApp#8284) bump workflows node-version to 24 (FreeTubeApp#8279) Translated using Weblate (Basque) Translated using Weblate (Vietnamese) Translated using Weblate (Hungarian) Translated using Weblate (Hungarian) Remove upperCase on various places (FreeTubeApp#8267) Translated using Weblate (French) Fix subscription count display on watch page with IV API (FreeTubeApp#8255) Specify window position in constructor for better wayland compatibility (FreeTubeApp#8238) Change profile icon to include lowercase letters (FreeTubeApp#8253) Translated using Weblate (German) Translated using Weblate (Chinese (Traditional Han script)) Translated using Weblate (Portuguese (Brazil))
Member
|
Single playlist view looks weird now (even with the screenshots you provided
|
4 tasks
7 tasks
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
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.
Pull Request Type
Related issue
closes #7104
Description
Long strings inside Playlists break the Channel Home and Playlists viewport and push content down. This PR wraps them correctly.
Screenshots
Channel Home Before
Channel Home After
Playlist Before
Playlist After
Testing
Go to a channel with very long playlist name (for example: mai_dq)
See that channel home page is not "pushed down" in the viewport
Scroll down to see that long playlist string is wrapped correctly
Click on View Playlist
See that long playlist string is wrapped correctly in playlist view