<div>Yes and no, in this case the problem is the <font style="font-family:monospace">beep</font> method</div><div>checks the aforementioned property. Someone even changed</div><div>this method to use <font style="font-family:monospace">PreferenceNG</font>, but did not bother to add</div><div>the property. If there is no <font style="font-family:monospace">SampleSound</font> class, it just sends</div><div><font style="font-family:monospace">primitiveBeep</font> to itself. I presume the later should just beep,</div><div>AFAIR it never worked on my Linux installation.</div><br><div>I see four unsolved issues with PreferenceNG (I have to</div><div>apologize, I did to review your code, I was busy getting out  a</div><div>Haver release before Haver's demo).</div><br><ol><li><div>It's difficult to install preferences for a package, at least without a special <font style="font-family:monospace">CodePackage</font> subclass.</div><div>We need something like <font style="font-family:monospace">PreferenceNG at: #somePreferenceSymbol ifAbsentInstall: [ ... ]</font></div></li><li><div>We should map the old Preferences code to PreferenceNG, this should provide the old default behavior.</div><div>This feature is important for packages the Cuis team can not change.</div></li><li><div>We have lot of unconverted packages in the Cuis repo.</div><div>Just try: <font style="font-family:monospace">find -iname '*.st' | xargs grep -l ' Preferences '</font>.</div></li><li><div>Implementing some sort of “world state object”, that should hold some sort of local preferences, is at least difficult,</div><div>because the whole logic is still at the class side. A <font style="font-family:monospace">PreferencesNGContainer</font> class, that has a singleton instance</div><div>bound to <font style="font-family:monospace">PreferencesNG</font> will solve that problem. Of course we need to move the class side methods of <font style="font-family:monospace">PreferenceNG</font></div><div>to the instance side of <font style="font-family:monospace">PreferencesNGContainer</font>.</div></li></ol><br><div>As I said, I am sorry that I mention this issues when it's almost to late.</div><br><br><div>Just my CHF 0.05,</div><br><div>Gerald</div><br><div class="gmail_quote_attribution">On May 29 2022, at 11:24 pm, Hilaire Fernandes via Cuis-dev <cuis-dev@lists.cuis.st> wrote:</div><blockquote><div> </div><div><div><font size="4">Hi Gerald,</font></div></div><br><div><font size="4">I do not know how to handle the soundsEnanbled preference because, if you have this option in the base image, without the sound package, one can set the preference to true but sound will not work. Am I right?</font></div><br><div><div><font size="4">If I am wrong, we can add it to the prefSystem method in PreferenceNG.</font></div></div><br><div><div><font size="4">Hilaire</font></div></div><br><div><br></div><br><div class="moz-cite-prefix"><div>Le 29/05/2022 à 10:35, Gerald Klix a écrit :</div></div><blockquote><br><div>I hate to point out that <font style="font-family:monospace">SystemDictionary>>#beep</font> is currently broken,</div><div>because <font style="font-family:monospace">PreferenceNG</font> lacks a <font style="font-family:monospace">#soundsEnabled</font> preference.</div><br><div><font style="font-family:monospace">@Juan and @Hilaire:</font></div><div><font style="font-family:monospace">How should one fix this properly?</font></div><ul><li><div>Provide a script that just adds this property</div></li><li><div>Add it to the <font style="font-family:monospace">PreferenceNG class>>#initialize</font> method.</div></li><li><div>Both of the above.</div></li></ul></blockquote><code><pre style="background-color:rgba(0,0,0,0.05);padding:0.2em 1em">-- <br>GNU Dr. Geo<br><a href="http://drgeo.eu" title="http://drgeo.eu">http://drgeo.eu</a><a href="http://blog.drgeo.eu" title="http://blog.drgeo.eu">http://blog.drgeo.eu</a></pre></code><div>--</div><div>Cuis-dev mailing list</div><div>Cuis-dev@lists.cuis.st</div><div>https://lists.cuis.st/mailman/listinfo/cuis-dev</div></blockquote>