Skip to content

Support for modifying source in compare clipboard editor #2007

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

SougandhS
Copy link
Contributor

For #1862 (comment)

follow-up to #1862

Supports modifying the source from compare editor itself

My.Movie.mp4

@SougandhS SougandhS marked this pull request as draft July 3, 2025 04:32
Copy link
Contributor

github-actions bot commented Jul 3, 2025

Test Results

 1 947 files  ±0   1 947 suites  ±0   1h 33m 49s ⏱️ - 3m 52s
 4 720 tests ±0   4 696 ✅ ±0   24 💤 ±0  0 ❌ ±0 
14 160 runs  ±0  13 993 ✅ ±0  167 💤 ±0  0 ❌ ±0 

Results for commit 443d921. ± Comparison against base commit 401d483.

♻️ This comment has been updated with latest results.

@SougandhS
Copy link
Contributor Author

One more thing I have to handle

@SougandhS
Copy link
Contributor Author

My.Movie2.mp4

@SougandhS SougandhS force-pushed the compareClipboardModify branch from ac391fe to 37ce6af Compare July 3, 2025 06:05
@SougandhS SougandhS marked this pull request as ready for review July 3, 2025 06:05
@SougandhS
Copy link
Contributor Author

Hi @iloveeclipse, I have added the missing modify functionality 👍

@SougandhS SougandhS force-pushed the compareClipboardModify branch 3 times, most recently from a0b6cee to a089065 Compare July 9, 2025 01:55
@vogella
Copy link
Contributor

vogella commented Jul 31, 2025

@iloveeclipse I think you suggest that @SougandhS adds this. Can you review?

@SougandhS SougandhS force-pushed the compareClipboardModify branch from cf4f3d5 to 443d921 Compare August 4, 2025 01:40
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.

3 participants