Issue 4789 in pharo: Improve the way morphs communicate with model

Status: FixProposed
Owner: [hidden email]

New issue 4789 by [hidden email]: Improve the way morphs communicate  
with model
http://code.google.com/p/pharo/issues/detail?id=4789

Implementation of a PluggableEventHandler which, if possible delegate event  
managing to the model

Attachments:
        AllowModelToBeNoticedByMorphs.1.cs  12.4 KB


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #1 on issue 4789 by [hidden email]: Improve the way morphs  
communicate with model
http://code.google.com/p/pharo/issues/detail?id=4789

I would not do that.
IHMO, the model should not have #mouseUpSelector or #click:from: ...



_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #2 on issue 4789 by [hidden email]: Improve the way morphs  
communicate with model
http://code.google.com/p/pharo/issues/detail?id=4789

I am working with Camillo on a better way to do that, by providing blocs or  
selectors :)

We are moving step by step ;)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Cc: [hidden email]

Comment #3 on issue 4789 by [hidden email]: Improve the way morphs  
communicate with model
http://code.google.com/p/pharo/issues/detail?id=4789

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #4 on issue 4789 by marianopeck: Improve the way morphs communicate  
with model
http://code.google.com/p/pharo/issues/detail?id=4789

I cannot open Nautilus because it uses PluggableEventHandler  which is NOT  
defined in the image. I am using the lastest one from Jenkis. Is this  
related to this issue?




_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: Closed
        Labels: Milestone-1.4

Comment #5 on issue 4789 by [hidden email]: Improve the way morphs  
communicate with model
http://code.google.com/p/pharo/issues/detail?id=4789

So I will add this as an intermediate step.

14139


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: FixProposed

Comment #6 on issue 4789 by [hidden email]: Improve the way morphs  
communicate with model
http://code.google.com/p/pharo/issues/detail?id=4789

A new cs to transfer event from the TextMorph to the PluggableTextMorph

(cs tested by Camillo)

Attachments:
        PluggableEventHandlerTextMorphFix.1.cs  1.1 KB


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Labels: Type-Feature

Comment #7 on issue 4789 by [hidden email]: Improve the way morphs  
communicate with model
http://code.google.com/p/pharo/issues/detail?id=4789

New class BlockEventHandler which you can provide a block or a  
model/selector for each event :)
@Mariano, if you want an annoying browser test:

in AbstractNautilusUI>>#buildCodePane change:
sourceTextArea eventHandler: (PluggableEventHandler model: self).
by:
sourceTextArea eventHandler: (BlockEventHandler new  
keyStrokeBlocOrSelector: [:t1 :t2 | Beeper primitiveBeep ]).
;)

Attachments:
        BlocEventHandler.2.cs  41.4 KB


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: Closed

Comment #8 on issue 4789 by [hidden email]: Improve the way morphs  
communicate with model
http://code.google.com/p/pharo/issues/detail?id=4789

in 14140


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker