Yes we should do that. let it and deprecate all the methods or remove the methods if they are in the superclass. Ben apparently monticello is bogus and the methods we moved from *soup to soup were removed from the class :( Stef On Jan 20, 2011, at 11:19 AM, Benjamin wrote:
Yes, code depends of this class, but this class inherits from PluggableTextMorph, so we can just remove everything in it, and just keep the name to keep the system working (and maybe move this class in the package Deprecated)
Ben
On Jan 20, 2011, at 11:09 AM, Tudor Girba wrote:
Yes, but I guess code still depends on this class. So, at the very least the class should still be preserved for one version.
Cheers, Doru
On 20 Jan 2011, at 11:05, Benjamin wrote:
In fact, PluggableShoutMorph shouldn't be used anymore, but PluggableTextMorph instead now that PluggableTextMorph handle the text styling :)
Ben
On Jan 20, 2011, at 9:34 AM, Tudor Girba wrote:
We probably need to remove that variable from PluggableShoutMorph. This variable does not exist in Pharo 1.2 12292.
Cheers, Doru
On 20 Jan 2011, at 09:11, laurent laffont wrote:
Other error: https://pharo-ic.lille.inria.fr/hudson/view/Pharo/job/Pharo%201.2/71/console
duplicatedVariableError: styler is already defined in PluggableTextMorph
Laurent
On Thu, Jan 20, 2011 at 8:38 AM, Tudor Girba <tudor.girba@gmail.com> wrote: Thanks Laurent and Alain for testing this.
Cheers, Doru
On 20 Jan 2011, at 08:21, Alain Plantec wrote:
the same here, it works. Now I can load Petit. Thanks Alain
Le 20/01/2011 08:08, laurent laffont a écrit :
It works ! It works !
Laurent
-- www.tudorgirba.com
"Live like you mean it."
-- www.tudorgirba.com
"Presenting is storytelling."
-- www.tudorgirba.com
"It's not how it is, it is how we see it."