Change profile icon to include lowercase letters - #8253
Merged
FreeTubeBot merged 2 commits intoNov 11, 2025
Merged
Conversation
FreeTubeBot
enabled auto-merge (squash)
November 9, 2025 22:40
github-actions
Bot
added
the
PR: waiting for review
efb4f5ff-1298-471a-8973-3d47447115dc
requested review from
a user,
MarmadileManteater,
PikachuEXE,
absidue and
kommunarr
November 9, 2025 22:48
Member
|
Note that with |
Member
|
Oops not changed here |
Member
Author
|
Good catch i somehow had that still sitting in my stash |
PikachuEXE
approved these changes
Nov 11, 2025
kommunarr
approved these changes
Nov 11, 2025
ghost
approved these changes
Nov 11, 2025
github-actions
Bot
removed
the
PR: waiting for review
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))
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
4 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
Description
I noticed this behavior when I had the
Hide profile pictures in commentssetting enabled. The profiles within the comment section always used a capital letter, even though I observed some profiles without pictures that used lowercase letters when the setting was turned off. After removing the.toUpperCase()from this setting, I realized we could apply the same approach to our own profiles. This is especially useful when two profiles use the same letter. Previously, the only way to distinguish them was by color, but now the appearance of the letter itself whether capitalized or not also serves as a differentiator.Screenshots
YT default profile icon when user didn't set profile picture:
Non capitalized icon
Capitalized icon
Before:
After:
Testing
Case A:
Hide profile pictures in commentsCase B:
Desktop