On Mon, May 21, 2018 at 09:17:24PM +0300, Denis Kudriashov wrote:
2018-05-21 21:14 GMT+03:00 Sven Van Caekenberghe <sven@stfx.eu>:
> On 21 May 2018, at 20:07, Guillermo Polito <guillermopolito@gmail.com> wrote: > > If I'm recalling well (which I might not :)) there is one thing (that I don't quite like) that prevents these two classes to be easily merged into one: their #defaultAction.
We need a good talk/discussion about default actions.
I am pretty confused by them, and my first reaction is that I don't like them, too much UI in there.
I am 100% with you. UI queries to users should be in UI.
Guille, thanks for the heads-up. I haven't thought about default actions until now, so I need to investigate before going much further. But having UI operations as part of the default action seems wrong for one of the core classes. My primary focus at the moment is to get the file attribute enhancements I've made integrated in to Pharo: https://pharo.fogbugz.com/f/cases/18279/isSymlink-seems-to-be-broken-on-Linu... but I'll continue to work on this at the same time. Cheers, Alistair