<div dir="ltr">Hi,<div>Thanks for reply.<br><div>I was thinking of using a block, but I was unsure how to use it :-)</div><div>I will see what I can come up with. </div></div><div>The functionality must work according to what color picker i selected in the preference for color picker also ;-)</div><div><br></div><div>Happy new year</div><div><br></div><div>Best,</div><div>Karl</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 1, 2016 at 2:43 PM, marcel.taeumel <span dir="ltr"><<a href="mailto:Marcel.Taeumel@hpi.de" target="_blank">Marcel.Taeumel@hpi.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Karl.<br>
<br>
I like it the non-modal way. It feels more immediate/direct. The button<br>
"close" does not suggest a "save operation." If we would make it modal, we<br>
would have to add another button to have "Save" and "Cancel".<br>
<br>
I don't like the way that the button/morph is used as a value holder for the<br>
color because we rely on that the button does not modify the color<br>
internally.<br>
<br>
To solve this problem, I would rather directly invoke the color picker in<br>
PBColorPreferenceView >> #changeColor:. For the new color picker, I would<br>
add some processing block for the target as an alternative to the hard-coded<br>
#color: message send. Like this:<br>
<br>
(NewColorPickerMorph with: [:newColor |<br>
aButton color: newColor.<br>
self preference preferenceValue: newColor.<br>
aButton label: newColor asString.<br>
self adjustLabelColor]) openInWorld.<br>
<br>
The color picker can evaluate the block with the new color as needed. The<br>
block closures the button and the preference view.<br>
<br>
Best,<br>
Marcel<br>
<br>
<br>
<br>
--<br>
View this message in context: <a href="http://forum.world.st/Delayed-preference-color-updating-tp4868940p4868960.html" rel="noreferrer" target="_blank">http://forum.world.st/Delayed-preference-color-updating-tp4868940p4868960.html</a><br>
Sent from the Squeak - Dev mailing list archive at Nabble.com.<br>
<br>
</blockquote></div><br></div>