E537 [Win32] Repair selection in WindowBuilder palette by ptziegler · Pull Request #704 · eclipse-windowbuilder/windowbuilder · GitHub
[go: up one dir, main page]

Skip to content

[Win32] Repair selection in WindowBuilder palette#704

Merged
ptziegler merged 1 commit intoeclipse-windowbuilder:masterfrom
ptziegler:win32-palette-selection
Jan 29, 2024
Merged

[Win32] Repair selection in WindowBuilder palette#704
ptziegler merged 1 commit intoeclipse-windowbuilder:masterfrom
ptziegler:win32-palette-selection

Conversation

@ptziegler
Copy link
Contributor

The deselection of the palette entries is not registered on Windows. While the entry can be selected, it remains selected even if other entries are chosen.

This is because the setCapture() method in Figure was removed and therefore those method calls in the PaletteComposite now reference the Control method instead. The reason this doesn't affect Linux is because here this is a no-op method.

Amends 360a0a6

The deselection of the palette entries is not registered on Windows.
While the entry can be selected, it remains selected even if other
entries are chosen.

This is because the setCapture() method in Figure was removed and
therefore those method calls in the PaletteComposite now reference the
Control method instead. The reason this doesn't affect Linux is because
here this is a no-op method.

Amends 360a0a6
@ptziegler ptziegler added this to the 1.15.0 milestone Jan 29, 2024
@github-actions
Copy link
github-actions bot commented Jan 29, 2024

Unit Test Results

         4 files           4 suites   1h 22m 14s ⏱️
  7 072 tests   6 883 ✔️ 189 💤 0
28 288 runs  27 528 ✔️ 760 💤 0

Results for commit ecfedd7.

♻️ This comment has been updated with latest results.

@ptziegler ptziegler merged commit 6ecd1dc into eclipse-windowbuilder:master Jan 29, 2024
@ptziegler ptziegler deleted the win32-palette-selection branch January 29, 2024 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

0