[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
Re: [riena-dev] Comments on setDefaultButton(...)
|
Quick heads up: after discussing this with Christian, we are opting
for the variant that uses the controller / snippet (b).
Elias.
On Fri, Jan 22, 2010 at 4:20 PM, Elias Volanakis
<elias@xxxxxxxxxxxxxxxxx> wrote:
> Hi riena-dev,
>
> I'm adding support for default buttons. We can either to it in the
> view (snippet a) or in the controller (snippet b).
>
> Snippet a - view:
> DefaultButtonManager dbm = new DefaultButtonManager(inputArea.getShell());
> dbm.addButton(inputArea, okButton); // Control, Button
>
> Snippet b - controller
> addDefaultAction(master2, actionApply); // IRidget, IActionRidet
>
>
> Doing it in the view has the benefit that it does _not_ depend on the
> navigation and the implementation is simpler. Drawback: conceptually I
> think this belongs sliglty more into the controller, plus if using a
> tool, this has to be added manually (when needed).
>
> Personally, I think doing it in the view so it can be used without
> Riena navigation is attractive. However I don't really have a strong
> preference.
>
> What do you think?
>
> Elias.
>
> --
> Elias Volanakis | Technical Lead | http://eclipsesource.com
> elias@xxxxxxxxxxxxxxxxx | +1 503 929 5537 | @evolanakis
>
--
Elias Volanakis | Technical Lead | http://eclipsesource.com
elias@xxxxxxxxxxxxxxxxx | +1 503 929 5537 | @evolanakis