Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 250624

Summary: Add Goto Line to compare contexts other than Compare Editor
Product: [Eclipse Project] Platform Reporter: Tomasz Zarna <tomasz.zarna>
Component: CompareAssignee: Platform-Compare-Inbox <platform-compare-inbox>
Status: CLOSED WONTFIX QA Contact: Tomasz Zarna <tomasz.zarna>
Severity: normal    
Priority: P4 CC: daniel_megert, pwebster, Szymon.Brandys
Version: 3.4   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard: stalebug

Description Tomasz Zarna CLA 2008-10-13 06:14:43 EDT
This is a follow up bug to bug 6823, where Goto Line was added to Compare Editor only.
Comment 1 Tomasz Zarna CLA 2008-10-14 04:09:47 EDT
From bug 6823, comment 26:
"[...] activating "Editing Text" for Compare Editor is [not] the best solution. Compare Editor can be opened not only for text comparisons. This fact makes bug 250624 [this bug] to rise in importance. The fix for it would probably activate the context for TextMergeViewer not CompareEditor."
Comment 2 Paul Webster CLA 2008-10-14 08:29:01 EDT
Is it that you need to redefine some keybindings in a compare only context?  Or do you mean your code should only activate the textScope context if you are working with TextMergeViewer (which also makes sense, I guess)?

PW
Comment 3 Tomasz Zarna CLA 2008-10-14 11:47:42 EDT
IMO the Comparing in an Editor context is to general, since the Compare Editor allows non-text input as well. For such input, actions like Goto Line don't make much sense. So I think I would go with the later, which is activating the textScope only when working with TextMergeViewer (or subclasses).

Comment 4 Szymon Brandys CLA 2009-03-06 05:32:49 EST
We haven't investigated the issue during 3.5 and we don't plan to address it now. I'm removing the target milestone but we'll set more realistic one when we have feedback from anyone interested in fixing this.
Comment 5 Eclipse Webmaster CLA 2019-09-06 16:16:20 EDT
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.
Comment 6 Eclipse Genie CLA 2022-01-12 16:59:35 EST
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're closing this bug.

If you have further information on the current state of the bug, please add it and reopen this bug. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.