Few Morphic questions/ issues

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

Few Morphic questions/ issues

S Krish

* CheckboxMorph does not implement #defaultBounds or take sensible defaults.

* PluggableTextFieldMorph  defaults to defaulBounds of MorphicModel, does not hideScrollBars.. both h and v.. ?, in effect is not defaulted to behave like a single line TextFieldMorph. 

* PluggableTextMorph has #defaultBounds of MorphicModel.. prefer a better bounds of a standard text morpsh typically flatter of 400@300 .. ?. Unless immediately initiated with #on:text:readSelection:menu, we need explicitly handling. I would suggest if we can look at initializing for the morph
PluggableTextMorph>>initialize

...
self setText: ''.
textMorph bounds: self defaultBounds.
self acceptOnFocusChange: true.
...
* SystemWindow >> openInWorld: aWorld
..
self bounds: (RealEstateAgent initialFrameFor: self world: aWorld).
...

    bounds should not be hardcoded.. to full extents. Use a more sensible flexible approach in all framework classes.

This perhaps a neat comment..!

openAsIsIn: aWorld
"Sad fixup for dodgy layout."

super openAsIsIn: aWorld.
self allMorphs do: [:m | m layoutChanged]

*  In Pharo 1.3 I do not have OBTextMorphSelection >> #debugIt



Reply | Threaded
Open this post in threaded view
|

Re: Few Morphic questions/ issues

Stéphane Ducasse
Thanks

What I suggest is produce some change set with the proposed fixes you have.
Morphic was written before new invoked initialize automatically and to of places would benefit from that.
Benjamin and alain helped already a lot to improve the situation.

Stef

On Feb 12, 2012, at 8:59 PM, S Krish wrote:

>
> * CheckboxMorph does not implement #defaultBounds or take sensible defaults.
>
> * PluggableTextFieldMorph  defaults to defaulBounds of MorphicModel, does not hideScrollBars.. both h and v.. ?, in effect is not defaulted to behave like a single line TextFieldMorph.
>
> * PluggableTextMorph has #defaultBounds of MorphicModel.. prefer a better bounds of a standard text morpsh typically flatter of 400@300 .. ?. Unless immediately initiated with #on:text:readSelection:menu, we need explicitly handling. I would suggest if we can look at initializing for the morph
> PluggableTextMorph>>initialize
>
> ...
> self setText: ''.
> textMorph bounds: self defaultBounds.
> self acceptOnFocusChange: true.
> ...
> * SystemWindow >> openInWorld: aWorld
> ..
> self bounds: (RealEstateAgent initialFrameFor: self world: aWorld).
> ...
>
>     bounds should not be hardcoded.. to full extents. Use a more sensible flexible approach in all framework classes.
>
> This perhaps a neat comment..!
>
> openAsIsIn: aWorld
> "Sad fixup for dodgy layout."
>
> super openAsIsIn: aWorld.
> self allMorphs do: [:m | m layoutChanged]
>
> *  In Pharo 1.3 I do not have OBTextMorphSelection >> #debugIt
>
> *
>
>


Reply | Threaded
Open this post in threaded view
|

Re: Few Morphic questions/ issues

S Krish
Will do so after a thorough test/ recheck on code by next weekend..

On Mon, Feb 13, 2012 at 1:59 AM, Stéphane Ducasse <[hidden email]> wrote:
Thanks

What I suggest is produce some change set with the proposed fixes you have.
Morphic was written before new invoked initialize automatically and to of places would benefit from that.
Benjamin and alain helped already a lot to improve the situation.

Stef

On Feb 12, 2012, at 8:59 PM, S Krish wrote:

>
> * CheckboxMorph does not implement #defaultBounds or take sensible defaults.
>
> * PluggableTextFieldMorph  defaults to defaulBounds of MorphicModel, does not hideScrollBars.. both h and v.. ?, in effect is not defaulted to behave like a single line TextFieldMorph.
>
> * PluggableTextMorph has #defaultBounds of MorphicModel.. prefer a better bounds of a standard text morpsh typically flatter of 400@300 .. ?. Unless immediately initiated with #on:text:readSelection:menu, we need explicitly handling. I would suggest if we can look at initializing for the morph
> PluggableTextMorph>>initialize
>
> ...
>       self setText: ''.
>       textMorph bounds: self defaultBounds.
>       self acceptOnFocusChange: true.
> ...
> * SystemWindow >> openInWorld: aWorld
> ..
>       self bounds: (RealEstateAgent initialFrameFor: self world: aWorld).
> ...
>
>     bounds should not be hardcoded.. to full extents. Use a more sensible flexible approach in all framework classes.
>
> This perhaps a neat comment..!
>
> openAsIsIn: aWorld
>       "Sad fixup for dodgy layout."
>
>       super openAsIsIn: aWorld.
>       self allMorphs do: [:m | m layoutChanged]
>
> *  In Pharo 1.3 I do not have OBTextMorphSelection >> #debugIt
>
> *
>
>