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

All of the UI pertaining to the new Chat Encryption feature are cut off on most common display resolutions, including 1920x1080 and 3456x2160. #200

Closed
akemin-dayo opened this issue Sep 26, 2022 · 10 comments
Assignees
Labels
bug Something isn't working confirmed Confirmed bug/approved suggestion

Comments

@akemin-dayo
Copy link

akemin-dayo commented Sep 26, 2022

Environment

Modloader: quilt-loader 0.17.4
Minecraft Version: 1.19.2, 1.19.1
No Chat Reports Version: 1.13.0

Bug Description

All of the UI pertaining to the new Chat Encryption feature are cut off on most common display resolutions, including 1920x1080 and 3456x2160.

This includes both "About Encryption" and "Encryption Settings".

It would probably be a good idea to decrease the amount of padding between each line, or something like that.

Screenshots ("About Encryption")

1920x1017@4x (Windowed mode on Windows)

framebuffer

1920x1080@4x (Fullscreen)

framebuffer

3456x1974@8x (Windowed mode on macOS)

framebuffer

3456x2160@8x (Fullscreen)

framebuffer

Screenshots ("Encryption Settings")

1920x1017@4x (Windowed mode on Windows)

framebuffer

1920x1080@4x (Fullscreen)

framebuffer

3456x1974@8x (Windowed mode on macOS)

framebuffer

3456x2160@8x (Fullscreen)

framebuffer

@akemin-dayo akemin-dayo added the bug Something isn't working label Sep 26, 2022
@akemin-dayo akemin-dayo changed the title The "About Encryption" screen is cut off on most common screen resolutions, including 1920x1080 and 3456x2160. The "About Encryption" screen is cut off on most common display resolutions, including 1920x1080 and 3456x2160. Sep 26, 2022
@akemin-dayo akemin-dayo changed the title The "About Encryption" screen is cut off on most common display resolutions, including 1920x1080 and 3456x2160. The contents of the "About Encryption" screen are cut off on most common display resolutions, including 1920x1080 and 3456x2160. Sep 26, 2022
@Aizistral Aizistral added confirmed Confirmed bug/approved suggestion 3rd party issue The issue is not related to this mod alone labels Sep 26, 2022
@akemin-dayo akemin-dayo changed the title The contents of the "About Encryption" screen are cut off on most common display resolutions, including 1920x1080 and 3456x2160. All of the UI pertaining to the new Chat Encryption feature are cut off on most common display resolutions, including 1920x1080 and 3456x2160. Sep 26, 2022
@Aizistral Aizistral added low priority and removed 3rd party issue The issue is not related to this mod alone labels Sep 26, 2022
@Aizistral
Copy link
Member

Does anyone really play with 4x scaling?

@akemin-dayo
Copy link
Author

akemin-dayo commented Sep 26, 2022

I do ;P

Myself aside, most people probably don't change the default UI scaling setting from "0" (Auto) anyway, which means Minecraft will always default to the largest UI scale possible.

@Aizistral
Copy link
Member

Fine, I'll find a way to fix this.

@akemin-dayo
Copy link
Author

Hmn, I'm noticing some odd behaviour even after 907df11.

If the UI scaling option is left at the default of 0 (Auto) for 3456x1974, it seems that specifically the "Encryption Settings" view does not go into "huge UI" mode. The "About Encryption" view appears to be unaffected, at least in my testing.

If you explicitly set a UI scale of 7 or 8 in the settings, then everything behaves as expected.

@Aizistral
Copy link
Member

This doesn't make any sense, both screens use absolutely identical calculation for whether or not they should go into huge mode. What NCR version did you test this with, and are you sure that auto is actually equal to 7 or 8?

@akemin-dayo
Copy link
Author

akemin-dayo commented Sep 28, 2022

What NCR version did you test this with

I've tested this with NCR 1.13.4 and 1.13.2.

are you sure that auto is actually equal to 7 or 8?

I have not programmatically verified this, but "Auto" does at least appear visually identical to both 7 and 8. (※ When the Unicode font is enabled, all odd-number UI scales become equivalent to their +1 value.)

both screens use absolutely identical calculation for whether or not they should go into huge mode

I was also thinking this was the case (given hugeGUI() in the code), but I… don't have any other explanation for what I'm seeing.

I'll try adding some debug logging for hugeGUI() later when I have time to see what's going on.


Screenshots

"About Encryption" displaying correctly at 0 UI scale

framebuffer

"Encryption Settings" somehow not going into huge UI mode at 0 UI scale

framebuffer

UI scale of 0 (Auto)

framebuffer

UI scale explicitly set to 8

framebuffer

… and now "Encryption Settings" works correctly when set explicitly to 8 (or 7 — the scaling is identical)

framebuffer

@Aizistral
Copy link
Member

Really weird. The only explanation I see is if screen height somehow is not calculated correctly in the case of settings screen, but I have no idea why it would go wrong with that screen specifically and not the other one.

@Aizistral
Copy link
Member

Ahhhhh I got it. I goes into huge mode, I just didn't check it correctly where line height is calculated...

@Aizistral
Copy link
Member

Test in v1.13.5, should be fixed.

@akemin-dayo
Copy link
Author

Can confirm, the issue is now fully fixed as of eafe2c8 / 1.13.5.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working confirmed Confirmed bug/approved suggestion
Projects
None yet
Development

No branches or pull requests

2 participants