On 01 Jul 2015, at 14:28, Thierry Goubier <thierry.goubier@gmail.com> wrote:Hi Marcus, Stef,2015-07-01 11:24 GMT+02:00 Marcus Denker <marcus.denker@inria.fr>:
> On 30 Jun 2015, at 19:21, stepharo <stepharo@free.fr> wrote:
>
> Hi guys
>
> does any of you use the AST code navigation?I haven't.I think we never enabled it because we could not use good key combinations��� the idea> Because we could remove it.
was to use ctrl-arrow keys for it���
I still think that AST based navigation is a very good idea, removing it now without having
ever used it for real is not good.Marcus, would that make sense to:* extract the core of the AST navigation as an API onto the RB AST (the ability to go up, down, left and right in a RB AST, basically), i.e. trying to preserve that way a bit some of the patterns solved by the AST navigation...* and reduce whatever is linked with the GUI / text morph components ?
Oh, looking at the code, the two aspects are implemented together, so it will be painfull to refactor. Looking a bit more at the code... I'm interested by the rationale for askForNodeSelectionFrom: