Skip to content

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
@github-actions
Copy link
Contributor

github-actions bot commented Jul 3, 2025

Test Results

 1 977 files  ±0   1 977 suites  ±0   1h 37m 12s ⏱️ + 8m 29s
 4 743 tests ±0   4 719 ✅ ±0   24 💤 ±0  0 ❌ ±0 
14 229 runs  ±0  14 047 ✅ ±0  182 💤 ±0  0 ❌ ±0 

Results for commit 9663a82. ± Comparison against base commit c905608.

♻️ 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
@vogella
Copy link
Contributor

vogella commented Dec 19, 2025

@SougandhS can you solve the merge conflict?

@SougandhS SougandhS force-pushed the compareClipboardModify branch from 443d921 to 28295b4 Compare December 19, 2025 08:39
@SougandhS
Copy link
Contributor Author

@SougandhS can you solve the merge conflict?

Done 👍

Support for modifying source in compare clipboard editor and Use
MultiPageEditorPart for getting active editor and in ClipboardReplace &
Clipboard Compare
@vogella vogella force-pushed the compareClipboardModify branch from 28295b4 to 9663a82 Compare January 12, 2026 10:29
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