8000 Use `isShowComplex()` when double-clicking on property by ptziegler · Pull Request #1323 · eclipse-windowbuilder/windowbuilder · GitHub
[go: up one dir, main page]

Skip to content

Use isShowComplex() when double-clicking on property#1323

Merged
ptziegler merged 1 commit intoeclipse-windowbuilder:masterfrom
ptziegler:property-use-is-show-complex
Dec 20, 2025
Merged

Use isShowComplex() when double-clicking on property#1323
ptziegler merged 1 commit intoeclipse-windowbuilder:masterfrom
ptziegler:property-use-is-show-complex

Conversation

@ptziegler
Copy link
Contributor

When using isComplex(), the property table attempts to flip a complex property, even though it doesn't contain any children. Due to how flipping is implemented, this then causes the property to lose its selection.

Using isShowComplex() avoids this problem.

When using `isComplex()`, the property table attempts to flip a complex
property, even though it doesn't contain any children. Due to how
flipping is implemented, this then causes the property to lose its
selection.

Using `isShowComplex()` avoids this problem.
@ptziegler ptziegler added this to the 1.23.0 milestone Dec 19, 2025
@github-actions
Copy link

Test Results

 2 720 files  + 7   2 720 suites  +7   10h 50m 31s ⏱️ - 1m 31s
 7 004 tests ± 0   6 946 ✅ + 1   58 💤 ±0  0 ❌  - 1 
28 016 runs  +36  27 784 ✅ +37  232 💤 ±0  0 ❌  - 1 

Results for commit 010cadb. ± Comparison against base commit 8c73a4a.

@ptziegler ptziegler merged commit 9d78b0c into eclipse-windowbuilder:master Dec 20, 2025
8 checks passed
@ptziegler ptziegler deleted the property-use-is-show-complex branch December 20, 2025 22:42
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