[etoys-dev] (SQ-796) Toolbar not restored after hiding

[Bert Freudenberg]

On 14.09.2010, at 16:18, K. K. Subramaniam wrote:

> On Tuesday 14 Sep 2010 3:09:27 pm Bert Freudenberg wrote:
>> On 14.09.2010, at 11:33, K. K. Subramaniam (JIRA) wrote:
>>> Thanks for the hint. I moved the delete into rebuildButtons after the
>>> owner check and checked a patch into MC.
>> 
>> That does sound more like a hack on top of a hack to me. We should rather
>> get rid of the silly preference. The "delete" does not belong in
>> "rebuildButtons", IMHO.
> Have patches to Sugar and System to get rid of this silly pref.

Thanks. This should go in the next release IMHO, not this one.

> System patch 
> needs a postscript to remove it from class var DictionaryOfPreferences. How do 
> I add a postscript to MC patch?

We have not been using MC postscripts to date, but rather adding it to the update stream.

The proper way would be to use #removePreference:.

- Bert -