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 926 files   1 926 suites   1h 28m 0s ⏱️
 4 745 tests  4 720 ✅  24 💤 0 ❌ 1 🔥
14 082 runs  13 899 ✅ 182 💤 0 ❌ 1 🔥

For more details on these errors, see this check.

Results for commit 28295b4.

♻️ 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?

Support for modifying source in compare clipboard editor and Use
MultiPageEditorPart for getting active editor and in ClipboardReplace &
Clipboard Compare
@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 👍

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