Community
Participate
Working Groups
This happens because in org.eclipse.jface.text.contentassist.CompletionProposalPopup.createProposalSelector(), the behavior of: fProposalTable.addSelectionListener(new SelectionListener() { @Override public void widgetSelected(SelectionEvent e) {} @Override public void widgetDefaultSelected(SelectionEvent e) { insertSelectedProposalWithMask(e.stateMask); } }); is OS-specific (see: https://bugs.eclipse.org/bugs/show_bug.cgi?id=508245). As on CompletionProposalPopup.verifyKey(), it checks for '\r', '\n' and passes along the event.stateMask, I think that the behavior should be the same when the pop-up has a focus (the reason why this is problematic for PyDev is explained in https://bugs.eclipse.org/bugs/show_bug.cgi?id=508245), so, my suggestion is changing the code to handle a KeyDown event and checking for '\r', '\n', as it does on the verifyEvent so that the same reasoning can be used whether the pop-up has focus or not... will provide a pull request for that -- unless someone sees this as a bad approach ;)
*** This bug has been marked as a duplicate of bug 287580 ***
Hi Markus, thank you for marking as a duplicate, still, that other bug doesn't seem to be the same thing (i.e.: On Windows the Ctrl/Shift is actually passed along, so, it's not the cause of the other bug reported in JDT). So, I'm reopening this bug (as the fix on this bug will *not* fix the issue reported at JDT related to the 'Ctrl' toggle override)... As a note, I have the intention of providing a fix for this one when I have time (but as I said, it won't fix that other issue as it's actually unrelated).
You're right it's not really a dup of bug 287580. Note that I posted bug 508245 comment 9 and reopened that bug. The missing event on GTK is an SWT bug that needs to be fixed there. That fix will make this request obsolete, so I'm closing it as a dup of that bug. *** This bug has been marked as a duplicate of bug 508245 ***
