| Summary: | Wrong autocomplete behavior with Emacs keymap | ||
|---|---|---|---|
| Product: | [WebTools] WTP Source Editing | Reporter: | Alexander Semenov <bohtvaroh> |
| Component: | wst.sse | Assignee: | 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
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. (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) 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. 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) (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? *** Bug 415823 has been marked as a duplicate of this bug. *** |