X Tutup
Skip to content

Clamp menus at the bottom of the screen.#109981

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
bruvzg:clamp_menui_h
Sep 20, 2025
Merged

Clamp menus at the bottom of the screen.#109981
Repiteo merged 1 commit intogodotengine:masterfrom
bruvzg:clamp_menui_h

Conversation

@bruvzg
Copy link
Member

@bruvzg bruvzg commented Aug 26, 2025

Fixes #109575

Before After
Screenshot 2025-08-26 at 11 22 02 Screenshot 2025-08-26 at 11 22 45

@bruvzg bruvzg added this to the 4.6 milestone Aug 26, 2025
@bruvzg bruvzg requested a review from a team as a code owner August 26, 2025 08:24
@KoBeWi
Copy link
Member

KoBeWi commented Aug 27, 2025

For some reason this does not work with the menus at the top of the editor.

image

Though it's unrelated to the issue.

@bruvzg
Copy link
Member Author

bruvzg commented Aug 27, 2025

For some reason this does not work with the menus at the top of the editor

Might be because I have set limit to 4 * menu button height to avoid to tiny menus (and top menu likely has bigger height than menu item).

@Repiteo Repiteo merged commit 74e869c into godotengine:master Sep 20, 2025
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Sep 20, 2025

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Editor Context menu items block underlying menu if bottom portion overflows screen height

3 participants

X Tutup