Review Request 115196: New declarative based assistant

[Aleix Pol]

On Tue, Jan 21, 2014 at 9:33 PM, Nicolai Hähnle-Montoro
<nhaehnle at gmail.com>wrote:

> Hello,
>
> I like the design of the screenshots.
>
> Not entirely off-topic because it is also related to the design: It
> reminded me of a question that often appeared in the back of my mind: Is it
> somehow possible to change the assistant such that I can still change my
> mind for some time after it appeared? The following happens quite
> frequently:
>
> 1. I edit a method definition.
> 2. The assistant suggesting to automatically adjust the method declaration
> appears.
> 3. My focus is elsewhere and I do something else first automatically.
> 4. The assistant disappears because I moved the cursor too far and I'm
> somewhat annoyed.
>
> I suppose such a change might involve having a backlog of assistant
> suggestions.
>
> Without knowing the code I guess this could be quite a bit of work. So
> perhaps it's just something to keep in the back of one's mind if the code
> is touched anyway at some point.
>
> Cheers,
> Nicolai
>
>
> On Tue, Jan 21, 2014 at 9:26 PM, Sven Brauch <svenbrauch at googlemail.com>wrote:
>
>>    This is an automatically generated e-mail. To reply, visit:
>> https://git.reviewboard.kde.org/r/115196/
>>   Review request for KDevelop.
>> By Sven Brauch.
>>  *Repository: * kdevplatform
>> Description
>>
>> A new proposal for what the assistant should look like. See screenshots.
>> Ideas welcome :)
>>
>>   Diffs
>>
>>    - shell/AssistantButton.qml (PRE-CREATION)
>>    - shell/CMakeLists.txt (f840acd08653d088cc597b098566f2a379f69968)
>>    - shell/assistantpopup.h (33a462aaa0186a524b0b04dcd3fc920a71219695)
>>    - shell/assistantpopup.cpp (c19a14982b0602882a5eed380c4c7aad6669c756)
>>    - shell/assistantpopup.qml (PRE-CREATION)
>>
>> View Diff <https://git.reviewboard.kde.org/r/115196/diff/>
>> File Attachments
>>
>>    - new assistant, bright colour scheme<https://git.reviewboard.kde.org/media/uploaded/files/2014/01/21/96c23c0e-106e-47e0-8f80-90303029e521__assistant_bright.png>
>>    - dark colour scheme<https://git.reviewboard.kde.org/media/uploaded/files/2014/01/21/d52643d2-3e0d-494b-bdd7-7beac3f4656c__assistant_dark.png>
>>    - alt pressed<https://git.reviewboard.kde.org/media/uploaded/files/2014/01/21/1df94b0b-d763-4aef-9182-ab113fcc3b1a__assistant_hl.png>
>>    - assistant, after triggering an action<https://git.reviewboard.kde.org/media/uploaded/files/2014/01/21/c118a296-778c-4cff-86a9-1ff63c2a3e48__assistant_trig.png>
>>
>>
>> _______________________________________________
>> KDevelop-devel mailing list
>> KDevelop-devel at kde.org
>> https://mail.kde.org/mailman/listinfo/kdevelop-devel
>>
>>
>
>
> --
> Lerne, wie die Welt wirklich ist,
> aber vergiss niemals, wie sie sein sollte.
>
> _______________________________________________
> KDevelop-devel mailing list
> KDevelop-devel at kde.org
> https://mail.kde.org/mailman/listinfo/kdevelop-devel
>
>
I'm unsure it would be a good idea to leave assistants after moving the
cursor, since the context of the assistant is defined by the cursor.

Aleix
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20140121/7743d404/attachment.html>