Skip to content

Conversation

@akurtakov
Copy link
Member

HeaderBar is part of the regular window size in Gtk 4 on Wayland BUT gtk_window_get_headerbar returns 0 if it is not set by the application but drawn internally by Gtk itself and thus it's size can not be queried. Add a "default" GtkHeaderBar explicitly for that and take its size into account when computing Shell size.

@akurtakov
Copy link
Member Author

This fixes initial size of Eclipse main window and dialogs, resizing is still broken. Fixes github.com//issues/2826

@github-actions
Copy link
Contributor

github-actions bot commented Dec 18, 2025

Test Results (linux)

   83 files   -  5     83 suites   - 5   10m 24s ⏱️ - 4m 10s
4 524 tests  - 33  4 311 ✅  - 26  212 💤  - 8  1 ❌ +1 
  175 runs   - 36    174 ✅  - 34    1 💤  - 2  0 ❌ ±0 

For more details on these failures, see this check.

Results for commit 124d5e5. ± Comparison against base commit d4f11be.

This pull request removes 33 tests.
org.eclipse.swt.tests.gtk.snippets.Bug336238_ShellSetBoundFailTest ‑ testSetBounds
org.eclipse.swt.tests.junit.Test_org_eclipse_swt_browser_Browser ‑ test_ClearAllSessionCookies
org.eclipse.swt.tests.junit.Test_org_eclipse_swt_browser_Browser ‑ test_OpenWindowListener_openHasValidEventDetails
org.eclipse.swt.tests.junit.Test_org_eclipse_swt_browser_Browser ‑ test_OpenWindowListener_open_ChildPopup
org.eclipse.swt.tests.junit.Test_org_eclipse_swt_browser_Browser ‑ test_OpenWindow_Progress_Listener_ValidateEventOrder
org.eclipse.swt.tests.junit.Test_org_eclipse_swt_browser_Browser ‑ test_StatusTextListener_hoverMouseOverLink
org.eclipse.swt.tests.junit.Test_org_eclipse_swt_browser_Browser ‑ test_VisibilityWindowListener_eventSize
org.eclipse.swt.tests.junit.Test_org_eclipse_swt_browser_Browser ‑ test_VisibilityWindowListener_multiple_shells
org.eclipse.swt.tests.junit.Test_org_eclipse_swt_browser_Browser ‑ test_get_set_Cookies
org.eclipse.swt.tests.junit.Test_org_eclipse_swt_browser_Browser_IE ‑ test_ClearAllSessionCookies
…

♻️ This comment has been updated with latest results.

@akurtakov akurtakov self-assigned this Dec 18, 2025
@akurtakov akurtakov force-pushed the master branch 6 times, most recently from 981eee6 to f184dc3 Compare December 23, 2025 08:55
@akurtakov akurtakov force-pushed the master branch 2 times, most recently from eb6b86b to 8e440a0 Compare December 29, 2025 08:48
HeaderBar is part of the regular window size  in Gtk 4 on Wayland BUT
gtk_window_get_headerbar returns 0 if it is not set by the application
but drawn internally by Gtk itself and thus it's size can not be
queried. Add a "default" GtkHeaderBar explicitly for that and take its
size into account when computing Shell size.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant