<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jan 18, 2016 at 1:24 AM, Sven Brauch <span dir="ltr"><<a href="mailto:mail@svenbrauch.de" target="_blank">mail@svenbrauch.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey all,<br>
<br>
with an eye on RR 126763, how do you imagine the future of the<br>
assistants? To me it is clear after some testing that the current way of<br>
having it in the completion, while having some some advantages, will not<br>
work out well just like that.<br>
<br>
There's two options: either revert to the old behaviour; or keep it in<br>
the completion list, but that requires solving of some issues from my<br>
point of view: having it open up interfers with navigation through the<br>
code, so one would need to think of a way to prevent that. Also to allow<br>
sensible automatic invocation, I think we'd need to decouple opening the<br>
upper completion popup from the normal completion list (which would make<br>
sense for other situations like calltips anyways btw).<br></blockquote><div><br></div><div>As the one responsible for leaving it in its current miserable state, I think it<br>should simply be reverted for now.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Maybe another idea to making it work with the completion list would be<br>
to make the upper part of the completion popup not keyboard-navigatable<br>
(and thus not catching the up/down arrow keys), and instead using the<br>
Alt+1 etc. shortcuts if applicable. The keyboard navigation in this<br>
widget never made sense for the use cases so far anyways.<br></blockquote><div><br></div><div>None of the uses of that top widget really shine with the CC-like navigation.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
What do you think? I'm actually quite undecided. At least for rename and<br>
add include, I like the workflow with the completion list better than<br>
the old workflow though.<br></blockquote><div><br></div><div>Even rename is unusable with current KTextEditor API -- try backspace. <br></div><div><br></div><div>I think having all the assistants in that general area would work if...<br><br></div><div>1. They had correct priority with respect to each other<br></div><div> -- Relatively easy<br></div><div>2. They didn't show up during automatic invocation<br></div><div> -- Doable<br></div><div>3. They had unintrusive notification, with active keyboard shortcuts<br></div><div> -- A bit of work<br></div><div>4. They were independent of the completion list (rather than reload it)<br></div><div> -- Requires new API from KTextEditor<br><br></div><div>(KTextEditor could offer to set an arbitrary widget which appears above the<br></div><div>line when invoked and otherwise has some unintrusive notification, we<br></div><div>could use it to replace all uses of "argument-hint completion")<br></div><div><br></div><div>Whether or not you like this particular idea of how it should work, it should<br></div><div>be pretty clear that there's lots to do in order to make anything of the sort<br></div><div>work. I hope someone wants to take it up, but for now we need to restore<br></div><div>the original assistant popup.<br></div><div><br></div><div>Cheers,<br></div><div><br></div><div>-Olivier JG<br><br></div><div><br></div></div></div></div>