Issue 4924 in pharo: BorderStyle dnu in BalloonRectangleMorph(BorderedMorph)>>borderStyle

Status: New
Owner: ----
Labels: Milestone-1.3 Type-Bug

New issue 4924 by [hidden email]: BorderStyle dnu in  
BalloonRectangleMorph(BorderedMorph)>>borderStyle
http://code.google.com/p/pharo/issues/detail?id=4924

Pharo image: 1.3 One-Click
Pharo core version: Pharo1.3 Latest update: #13307
Virtual machine used: MacOSX i386 timestamp: 2011 Aug 31 15:36
Class browser used if applicable:  OBSystemBrowserAdaptor

Steps to reproduce:
1. BalloonRectangleMorph new openInWorld
2. drag the resulting morph

You get a dnu, because BorderedMorph>>borderStyle sends BorderStyle a  
perform: borderColor, but borderColor is a GradientFillStyle rather than a  
symbol.

Unfortunately I don't know enough about morphic to figure it out.


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

Updates:
        Status: Accepted

Comment #1 on issue 4924 by [hidden email]: BorderStyle dnu in  
BalloonRectangleMorph(BorderedMorph)>>borderStyle
http://code.google.com/p/pharo/issues/detail?id=4924

Balloon is not really used right now... so I am sure it does not work.

Is it important to fix this for 1.3? Or can we move it to 1.4?


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

Comment #2 on issue 4924 by [hidden email]: BorderStyle dnu in  
BalloonRectangleMorph(BorderedMorph)>>borderStyle
http://code.google.com/p/pharo/issues/detail?id=4924

I personally don't care, but the code is actually in BorderedMorph, so
I'm not sure if this has wider implications.

../Dave


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

Updates:
        Labels: Milestone-1.4

Comment #3 on issue 4924 by [hidden email]: BorderStyle dnu in  
BalloonRectangleMorph(BorderedMorph)>>borderStyle
http://code.google.com/p/pharo/issues/detail?id=4924

(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 4924 by [hidden email]: BorderStyle dnu in  
BalloonRectangleMorph(BorderedMorph)>>borderStyle
http://code.google.com/p/pharo/issues/detail?id=4924

we should get rid of borderedmorph class.
Morphs could have a "borderstyle" property, where all border-specific  
things can be defined: border width, color etc.

I would understand if borderedmorph would draw things differently, then  
apparently you need to have a subclass. But since it doesn't , and a border  
is more visual style property than related to the way how morph  
functioning , i don't see a compelling reason why BorderedMorph should  
exist in separate class.


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

Updates:
        Status: WontFix
        Labels: -Milestone-1.3

Comment #5 on issue 4924 by [hidden email]: BorderStyle dnu in  
BalloonRectangleMorph(BorderedMorph)>>borderStyle
http://code.google.com/p/pharo/issues/detail?id=4924

ok, we deprecate the BalloonRectangleMorph in 1.4


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