On Mon, May 21, 2018 at 09:17:24PM +0300, Denis Kudriashov wrote: > > 2018-05-21 21:14 GMT+03:00 Sven Van Caekenberghe <[email protected]>: > > > > > On 21 May 2018, at 20:07, Guillermo Polito <[email protected]> > 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-Linux but I'll continue to work on this at the same time. Cheers, Alistair
