mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-07-24 17:35:35 +02:00
Code Issues Wiki Activity

Bug 141973: Bad cursor position after auto-completion

Browse source 
The line in DOMCompletionContributor.handleVariable() :
    proposal.setCursorPosition(repString.length() - 1);
seems to be copied from previous method handleFunction().
It's reasonable for function (put cursor INSIDE brackets
after function name), but useless for variable (it has
no trailing brackets, so cursor will stop 1 symbol left
from the end on variable name).

So mentioned line is removed: default cursor position
(set inside of CCompletionProposal constructor) is OK.

          Oleg Krasilnikov <oleg.krasilnikov@intel.com>
This commit is contained in:
Oleg Krasilnikov 2006-06-05 13:42:58 +00:00
parent 49f74f1cb2
commit 86d252bd88
1 changed files with 0 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/DOMCompletionContributor.java
View file

@ -216,9 +216,6 @@ public class DOMCompletionContributor implements ICompletionContributor {
int repLength = completionNode.getLength();
int repOffset = offset - repLength;
CCompletionProposal proposal = new CCompletionProposal(repString, repOffset, repLength, null, dispString, idString, 1, viewer);
proposal.setCursorPosition(repString.length() - 1);
proposals.add(proposal);
}