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

Tiny paddings in popup windows #89718

Closed
passivestar opened this issue Mar 20, 2024 · 9 comments · Fixed by #92647
Closed

Tiny paddings in popup windows #89718

passivestar opened this issue Mar 20, 2024 · 9 comments · Fixed by #92647
Milestone

Comments

@passivestar
Copy link
Contributor

Tested versions

Reproducible in 4.2.1 and 4.3

System information

macOS 14.2.1 - Vulkan (Forward+) - integrated Apple M1 Max - Apple M1 Max (10 Threads)

Issue description

There's basically no space around popup window content on macOS no matter what editor settings I try, this is the default theme:

image

This is on windows:

image

The conversation about this started here

P.S. The default padding could also be increased by ~50%, because even on windows it look a bit too packed

Steps to reproduce

N/A

Minimal reproduction project (MRP)

N/A

@Calinou
Copy link
Member

Calinou commented Mar 20, 2024

@akien-mga
Copy link
Member

CC @bruvzg

@bruvzg
Copy link
Member

bruvzg commented Apr 18, 2024

I can't see any difference in the padding on macOS and Windows. What's your editor scale and macOS screen scale?

@passivestar
Copy link
Contributor Author

passivestar commented Apr 18, 2024

I can't see any difference in the padding on macOS and Windows. What's your editor scale and macOS screen scale?

Default system screen scale, 16 inch macbook (3456x2234), in Godot I usually use 1.6 but it doesn't look any different at 2.0 (which is the default)

@passivestar
Copy link
Contributor Author

passivestar commented May 29, 2024

I just noticed @adamscott doesn't have paddings in his popups too: #90660

image

They need more space:

image

So this issue isn't strictly macOS-specific

@coppolaemilio
Copy link
Member

@adamscott and @passivestar are you using a custom theme at all? Does this happen also in the default one?

@passivestar
Copy link
Contributor Author

passivestar commented May 29, 2024

@coppolaemilio we do but I have the same problem with the default theme (the first screenshot I attached on top is using the default one). We both seem to be using a high dpi display though

Here's the default theme:

image

Same 4 pixels:

image

@coppolaemilio
Copy link
Member

So the paddings might not be getting multiplied by the rendering scale 🤔

@passivestar
Copy link
Contributor Author

passivestar commented May 29, 2024

So the paddings might not be getting multiplied by the rendering scale 🤔

Probably yes

But it'd also not be enough. Multiplying 4px by my editor scale 1.5 would give me 6px which is practically the same

I had to lift by 12 pixels here (for a total of 16) for it to look not scuffed:

image

For reference: both Editor Settings and Project Settings windows have exactly that - 16px of space on my screen. I suggest using the same value for all of the popups

image

@passivestar passivestar changed the title [macOS] No paddings in popup windows Tiny paddings in popup windows May 29, 2024
@AThousandShips AThousandShips added this to the 4.3 milestone Jun 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants