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

Buttom of item text lables are being cut off for CJK font #6889

Closed
teddy516 opened this issue Jan 10, 2024 · 15 comments
Closed

Buttom of item text lables are being cut off for CJK font #6889

teddy516 opened this issue Jan 10, 2024 · 15 comments

Comments

@teddy516
Copy link

屏幕截图 2024-01-10 165440

The previous version 1.4.1 text display was perfect, but the new version has this problem, I hope it can be fixed, thank you.

@AJenbo
Copy link
Member

AJenbo commented Jan 10, 2024

Can you please be more specific about what the error is, maybe show a comparison

@teddy516
Copy link
Author

屏幕截图 2024-01-10 205801

Compare text labels for potions on the ground

@DakkJaniels
Copy link
Contributor

Are you saying the bottom of the text is getting cut off? Probably would have been helpful if you used the same potions.

@AJenbo AJenbo changed the title The text display is incomplete Buttom of item text lables are being cut off Jan 10, 2024
@AJenbo AJenbo changed the title Buttom of item text lables are being cut off Buttom of item text lables are being cut off for CJK font Jan 10, 2024
@teddy516
Copy link
Author

屏幕截图 2024-01-10 211746
屏幕截图 2024-01-10 212303

@teddy516
Copy link
Author

屏幕截图 2024-01-10 213528
In addition, the font size displayed in the status bar at the bottom is not appropriate.

@AJenbo
Copy link
Member

AJenbo commented Jan 10, 2024

Please give us an idea of what is expected, most of us do not read cjk, so "not right" is practically impossible for us to fix with out saying exactly what needs to be changed for it to be corrected.

@teddy516
Copy link
Author

teddy516 commented Jan 10, 2024

I'm sorry, I'm just an ordinary player, I don't know what the specific reason is, I can't understand why the text will be abnormal when I update the new version, I only hope to restore the normal text display state of 1.4.1, if I know programming, maybe I can find out the difference between the two versions of the font, so I can only ask the professionals here. Maybe it could be done by resizing the font or changing it to another font?

@DakkJaniels
Copy link
Contributor

It's not about the programming. We (or at least I) don't understand what the characters are supposed to look like.

That being said, I think it's more clear in the potion comparison that the bottom of the characters are getting cut off in 1.5.1.

@DakkJaniels
Copy link
Contributor

DakkJaniels commented Jan 10, 2024

If you look at the two pictures, you can see in 1.4.1 that the text is going beyond the bottom of the box and still rendering, but in 1.5.1 the text is cut off a pixel or two from the bottom of the box:

image

Edit, it also looks like the top of the text in 1.5.1 is rendering outside the top of the box which matches 1.4.1, I think.

@AJenbo
Copy link
Member

AJenbo commented Jan 10, 2024

Maybe it could be done by resizing the font or changing it to another font?

The CJK font is already the minimal size possible for representing legible CJK letters.

If you look at the two pictures, you can see in 1.4.1 that the text is going beyond the bottom of the box and still rendering, but in 1.5.1 the text is cut off a pixel or two from the bottom of the box:

Sounds like the box simply needs to be higher when displaying CJK texts

@AJenbo
Copy link
Member

AJenbo commented Jan 10, 2024

@teddy516 like Dakk said we do not expect you to tell us what technical changes are needed, but you do need to explain to us what "appropriate" looks like.

屏幕截图 2024-01-10 213528 In addition, the font size displayed in the status bar at the bottom is not appropriate.

I have no idea what isn't appropriate about this, you might as well have shown me The Monalisa and said it wasn't done right by the artist. I don't know if she was actually supposed to look like an old man or there should be snow in the background.

@teddy516
Copy link
Author

good
屏幕截图 2024-01-11 150508

bad
屏幕截图 2024-01-11 150453

@glebm
Copy link
Collaborator

glebm commented Jan 11, 2024

I've sent a fix for the item labels in #6894. With the fix it is better than 1.4.1 because the label no longer overflows the box, instead the box is made larger for CJK.

The info box (panel at the bottom of the screen) is a known issue and was similar to this in 1.4.1 as well (there simply isn't enough space) but I've made it slightly less bad in #6895

Basically what changed from 1.4 to 1.5 is that the text is now clipped to the box it's drawn in. This bugfix exposed some boxes that were too small.

@glebm
Copy link
Collaborator

glebm commented Jan 11, 2024

Item labels are now fixed, info box issue is known and is tracked in #3549

@glebm glebm closed this as completed Jan 11, 2024
@AJenbo
Copy link
Member

AJenbo commented Jan 11, 2024

Feel free to put it in the 1.5 branch as well, I have a cold atm so will be a bit spotty atm

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

No branches or pull requests

4 participants