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

Bug 315549

Summary: Wrong autocomplete behavior with Emacs keymap
Product: [WebTools] WTP Source Editing Reporter: Alexander Semenov <bohtvaroh>
Component: wst.sseAssignee: wst.sse <wst.sse-inbox>
Status: NEW --- QA Contact: Nick Sandonato <nsand.dev>
Severity: normal    
Priority: P3 CC: fangzhenzyou, rakes123
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Linux   
Whiteboard:

Description Alexander Semenov CLA 2010-06-03 06:25:20 EDT
Build Identifier: 20100527-0614

Set eclipse keymap to Emacs.
Issue an autocomplete window while editing an XML or HTML file.
Press C-n or C-p keys.
Instead of moving selection inside the autocomplete window it moves cursor in the source file.
Notice, that this feature works fine while editing Java code.

Reproducible: Always

Steps to Reproduce:
1. Set eclipse keymap to Emacs.
2. Issue an autocomplete window while editing an XML or HTML file.
3. Press C-n or C-p keys.
4. Notice the incorrect behavior.
Comment 1 Rakesh CLA 2010-07-28 06:40:19 EDT
I tried it on I20100608-0911 build , on both xml and html files.

C-n moves cursor to next line (keeping cursor same if possible)
C-p moves cursor to line above.
I saw same behaviour for java files , and it seems to be correct.Please let me know if i missed something.
Comment 2 Rakesh CLA 2010-07-28 06:42:33 EDT
(In reply to comment #1)
> I tried it on I20100608-0911 build , on both xml and html files.
> 
> C-n moves cursor to next line (keeping cursor same if possible)
> C-p moves cursor to line above.
> I saw same behaviour for java files , and it seems to be correct.Please let me
> know if i missed something.

small correction  *(keeping cursor's horizontal position same if possible)
Comment 3 Alexander Semenov CLA 2010-07-28 08:05:22 EDT
For Java files these keybindings move selection inside the autocomplete window (you can check it yourself) but for XML/HTML files cursor is moved in the source file but not in the autocomplete popup window.
Comment 4 Rakesh CLA 2010-08-02 07:29:36 EDT
SourceViewer has declared getContentAssistantFacade() as final , which is provided by ISourceViewerExtension4 interface. This interface is responsible for keybindings used in ContentAssist proposals.We will see this behavior unless we override this function(or copy/paste whole bunch of functionality)
Comment 5 Rakesh CLA 2010-08-27 03:40:20 EDT
(In reply to comment #4)
> SourceViewer has declared getContentAssistantFacade() as final , which is
> provided by ISourceViewerExtension4 interface. This interface is responsible
> for keybindings used in ContentAssist proposals.We will see this behavior
> unless we override this function(or copy/paste whole bunch of functionality)
Should we request SourceViewer team to remove this function as final?
Comment 6 Nick Sandonato CLA 2013-08-26 12:43:46 EDT
*** Bug 415823 has been marked as a duplicate of this bug. ***