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 losing title of windows with a 3 or 4 bytes first character #758

Merged
merged 1 commit into from
Jul 19, 2023

Conversation

cwendling
Copy link
Member

Fixes #757.

Copy link
Member

@lukefromdc lukefromdc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Builds clean, and fixes the issue though with my fonts I only get the missing-character symbols. Without this the space stays entirely empty however, so this should confirm it works. Someone with the appropriate fonts installed ideally would also test this, but I doubt the characters themselves would change with this as opposed to whether they are displayed

@lukefromdc lukefromdc requested a review from a team July 18, 2023 16:13
@cwendling
Copy link
Member Author

@lukefromdc it shouldn't change anything, but here I get proper characters (well, I see Chinese characters, and assume it's the right ones :)) and it works

@raveit65
Copy link
Member

@cloudfish Can you please test PR

@raveit65
Copy link
Member

I have no idea what's wrong with https://zh.wikipedia.org/ on my fedora system. Chinese symbols are in the middle of the window title. Without a screenshot from reporter i would say it's debian specific :)

@raveit65
Copy link
Member

I can confirm now the issue, sadly the bad commit is in latest 1.26. release.
I will test PR, but without a clear confirmation for PR from someone who use Chinese symbols, i don't like to do a new release for one commit to avoid breaking stable 1.26 branch again.

@cwendling
Copy link
Member Author

@raveit65 hopefully @cloudfish will be able to test, but this is a very simple fix I am confident in. If you confirm it fixes the Firefox example for you as well we should be good. A maybe more important test is whether it doesn't break #749 -- but my tests with KCalc tell me it doesn't -- and the logic I see agrees.

Copy link
Member

@raveit65 raveit65 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirmed that it fixes the issue
and it doesn't break previous fix for running applications as superuser.

@raveit65
Copy link
Member

If it is urgent Debian maintainer can use it as patch for his build.
Same what i do in fedora when it is urgent.
I like to test things a while before they go in stable.
Btw. everybody from the team can do new releases. This is not only in my hand ;)

@raveit65 raveit65 merged commit 32bfada into master Jul 19, 2023
3 checks passed
@raveit65 raveit65 deleted the utf8-title branch July 19, 2023 10:59
@cwendling
Copy link
Member Author

@mate-desktop/core-team can I cherry-pick this in 1.26? We have the breaking change there, and it's affecting distros that might or might not know about it.

@cwendling
Copy link
Member Author

Cherry-picked as a938884

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Window title disappears when the first character is Chinese
3 participants