> On 01 Jul 2015, at 14:28, Thierry Goubier <thierry.goub...@gmail.com> wrote: > > Hi Marcus, Stef, > > 2015-07-01 11:24 GMT+02:00 Marcus Denker <marcus.den...@inria.fr > <mailto:marcus.den...@inria.fr>>: > > > On 30 Jun 2015, at 19:21, stepharo <steph...@free.fr > > <mailto:steph...@free.fr>> wrote: > > > > Hi guys > > > > does any of you use the AST code navigation? > > I haven't. > > > Because we could remove it. > > I think we never enabled it because we could not use good key combinations… > the idea > 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 ? >
Yes, sounds good... > 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: >
