Skip to content

Remove Ctrl C accelerator in Copy Variable/Expression#2671

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

Remove Ctrl C accelerator in Copy Variable/Expression#2671
SougandhS wants to merge 1 commit into
eclipse-platform:masterfrom
SougandhS:bugFix

Conversation

@SougandhS
Copy link
Copy Markdown
Contributor

@SougandhS SougandhS commented May 23, 2026

Removes cmd+c accelerator in Copy Variable & Copy Expression Context of
Expressions View and Variables View

Screenshot 2026-05-23 at 9 21 13 PM

Fixes : #2670

Comment thread debug/org.eclipse.debug.ui/plugin.xml Outdated
@ewillink
Copy link
Copy Markdown

-1

No. The legacy Ctrl-C behaviour was "Copy Variables" which is now "Copy Row".

It is "Copy Variable" that is a new functionality and so must have a new accelerator.

("Copy Row" is a better spelling of "Copy Variables").

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 23, 2026

Test Results

    54 files  ±0      54 suites  ±0   34m 51s ⏱️ - 1m 19s
 4 667 tests ±0   4 645 ✅ ±0   22 💤 ±0  0 ❌ ±0 
11 895 runs  ±0  11 742 ✅ ±0  153 💤 ±0  0 ❌ ±0 

Results for commit 5939433. ± Comparison against base commit 71fc717.

♻️ This comment has been updated with latest results.

@SougandhS SougandhS force-pushed the bugFix branch 2 times, most recently from 9598978 to ac54b67 Compare May 23, 2026 15:26
@SougandhS
Copy link
Copy Markdown
Contributor Author

-1

No. The legacy Ctrl-C behaviour was "Copy Variables" which is now "Copy Row".

It is "Copy Variable" that is a new functionality and so must have a new accelerator.

("Copy Row" is a better spelling of "Copy Variables").

Will add a new shortcut for Copy Variable & Copy Expression context in 4.41 M1. Right now removed the duplicate accelerators

@SougandhS SougandhS changed the title Remove Ctrl C accelerator in Copy Row Remove Ctrl C accelerator in Copy Variable/Expression May 23, 2026
@SougandhS SougandhS requested a review from merks May 23, 2026 16:20
Copy link
Copy Markdown
Contributor

@merks merks left a comment

Choose a reason for hiding this comment

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

I think there are some pointless overrides.

Removes cmd+c accelerator in Copy Variable & Copy Expression Context of
Expressions View and Variables View

Fixes : eclipse-platform#2670
@SougandhS
Copy link
Copy Markdown
Contributor Author

I think there are some pointless overrides.

You're right, that overrides was unnecessary. - Removed it 👍

@SougandhS SougandhS requested a review from merks May 25, 2026 01:37
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.

Regression: Confusing Ctrl C in Debug View

3 participants