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

Interpret labels as UTF-8 in testFitStringInTextBox #22

Closed
wants to merge 1 commit into from
Closed

Interpret labels as UTF-8 in testFitStringInTextBox #22

wants to merge 1 commit into from

Conversation

madscientist159
Copy link
Contributor

This fixes labels not showing up when certain characters,
e.g. "à", were used in those labels.

This fixes issue #4

Signed-off-by: Timothy Pearson [email protected]

This fixes labels not showing up when certain characters,
e.g. "à", were used in those labels.

Signed-off-by: Timothy Pearson <[email protected]>
@derickr
Copy link
Member

derickr commented Jun 29, 2016

Would you be able to add a test case?

@madscientist159
Copy link
Contributor Author

I'm not entirely sure how this would be implemented as a test case. Is there a function that checks for the existence of rendered text on a graph?

@derickr
Copy link
Member

derickr commented Jul 11, 2016

No, but I think there are tests that check the output of the bits that orient the items on the graph. Perhaps @kore can chime in on that?

(And sorry for the late reply, I was on holiday)

@derickr
Copy link
Member

derickr commented Aug 17, 2016

Ping @kore

@beberlei
Copy link
Member

I tried to reproduce this on Linux by adding appropriate tests that compare image similarity but no avail, without /u it already works fine with all the examples given in #4 - I am inclined to just merge this.

derickr added a commit that referenced this pull request Feb 10, 2022
@derickr
Copy link
Member

derickr commented Feb 10, 2022

Merged (After rebase), so closing.

@derickr derickr closed this Feb 10, 2022
@beberlei
Copy link
Member

We had to revert this as not all tests pass with this change, specifically when the text is not UTF-8 but ISO-8859-1, then using '/u' will fail.

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.

3 participants