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

egui: Fix incorrect handling of item spacing in Window title bar #3995

Merged
merged 1 commit into from
Feb 7, 2024

Conversation

varphone
Copy link
Contributor

@varphone varphone commented Feb 7, 2024

Currently, the window title bar does not correctly handle the item_spacing and window_margin.

  • Video snapshot on changing the item_spacing and window_margin (master)
egui-demo-app-incorrect.webm
  • Video snapshot on changing the item_spacing and window_margin (feature/fix-window-title-bar)
egui-demo-app-fixed.webm

@varphone varphone changed the title Fixed window title bar incorrect handling spacing egui: Fixed window title bar incorrect handling spacing Feb 7, 2024
Copy link
Owner

@emilk emilk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome; thanks for the videos too!

@emilk emilk added bug Something is broken egui labels Feb 7, 2024
@emilk emilk merged commit 377f86e into emilk:master Feb 7, 2024
20 of 21 checks passed
emilk pushed a commit that referenced this pull request Feb 10, 2024
Currently, the window title bar does not correctly handle the
`item_spacing` and `window_margin`.

* Video snapshot on changing the `item_spacing` and `window_margin`
(master)


[egui-demo-app-incorrect.webm](https://github.com/emilk/egui/assets/1274171/a4e32d43-69b5-44be-a4c3-97b0533147ca)

* Video snapshot on changing the `item_spacing` and `window_margin`
(feature/fix-window-title-bar)


[egui-demo-app-fixed.webm](https://github.com/emilk/egui/assets/1274171/42bbc210-a1f3-4e0a-ab71-d068e58e0e35)
@emilk emilk changed the title egui: Fixed window title bar incorrect handling spacing egui: Fix incorrect handling of item spacing in Window title bar Mar 26, 2024
hacknus pushed a commit to hacknus/egui that referenced this pull request Oct 30, 2024
Currently, the window title bar does not correctly handle the
`item_spacing` and `window_margin`.

* Video snapshot on changing the `item_spacing` and `window_margin`
(master)


[egui-demo-app-incorrect.webm](https://github.com/emilk/egui/assets/1274171/a4e32d43-69b5-44be-a4c3-97b0533147ca)

* Video snapshot on changing the `item_spacing` and `window_margin`
(feature/fix-window-title-bar)


[egui-demo-app-fixed.webm](https://github.com/emilk/egui/assets/1274171/42bbc210-a1f3-4e0a-ab71-d068e58e0e35)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is broken egui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants