Skip to content

Fix broken launch config dialog UI after unsaved-changes prompt on macOS - #3003

Open
SougandhS wants to merge 1 commit into
eclipse-platform:masterfrom
SougandhS:FIxLaunchDial
Open

SougandhS wants to merge 1 commit into
eclipse-platform:masterfrom
SougandhS:FIxLaunchDial

Conversation

@SougandhS

Copy link
Copy Markdown
Contributor

On macOS the tree selects on mouse down, and opening the modal "unsaved changes" prompt at that point corrupts the tree. Skip selection changes while the mouse is pressed and handle them on release.

Before :

Screen.Recording.2026-10-09.at.10.52.07.AM.mov

After :

Screen.Recording.2026-10-09.at.10.52.58.AM.mov

Fixes eclipse-jdt/eclipse.jdt.debug#909

@github-actions

github-actions Bot commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

Test Results

    54 files  ±0      54 suites  ±0   31m 56s ⏱️ - 4m 27s
 4 909 tests ±0   4 887 ✅ ±0   22 💤 ±0  0 ❌ ±0 
12 618 runs  ±0  12 464 ✅ ±0  154 💤 ±0  0 ❌ ±0 

Results for commit 46235aa. ± Comparison against base commit 1b7b555.

♻️ This comment has been updated with latest results.

@SougandhS
SougandhS force-pushed the FIxLaunchDial branch 2 times, most recently from a94b671 to 55e562f Compare October 11, 2026 17:02

@merks merks left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I’m looking on my phone. I likely can and a better job on a bigger screen.

@SougandhS
SougandhS force-pushed the FIxLaunchDial branch 2 times, most recently from 0ead9cb to 5a30600 Compare October 11, 2026 17:45
@SougandhS
SougandhS requested a review from merks October 13, 2026 01:07
@SougandhS
SougandhS force-pushed the FIxLaunchDial branch 2 times, most recently from fa06f47 to a8fa439 Compare October 15, 2026 07:44
@SougandhS

Copy link
Copy Markdown
Contributor Author

Just rebased on master 👍 ..

@SougandhS

Copy link
Copy Markdown
Contributor Author

Hi @merks, could you please re-check this PR ?

@merks merks left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It looks okay. I haven't tested though...

@merks
merks requested a review from iloveeclipse October 18, 2026 11:11
@SougandhS

Copy link
Copy Markdown
Contributor Author

Hi @iloveeclipse, can you have a look at this ?

On macOS the tree selects on mouse down, and opening the modal "unsaved
changes" prompt at that point corrupts the tree. Skip selection changes
while the mouse is pressed and handle them on release.

Fixes eclipse-jdt/eclipse.jdt.debug#909
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.

editing env vars in launch configs causes UI issues

2 participants