On Feb 10, 2012, at 2:45 PM, Ricardo Moran wrote:
Hi again, I've been working on the "decimal places" option and now I don't feel is a good idea because we would still have the "decimal places..." menu item in system-defined slots. I feel this wouldn't be very consistent. Maybe the dialog with just name & type would be good enough.
Arguably yes... However, since you've already done the work to have the decimal-places element come and go in the panel, depending on the type chosen, and since its presence in the panel helps make users conscious of the "decimal places" feature, which they might otherwise not be aware of, maybe we should keep it there...
In any case, this is an excellent and timely and really welcome piece of work, Richo -- yay!
Any comments from Ed Team about this?
-- Scott
In any case, I attached the new change set for you to try.
Cheers, Richo
On Fri, Feb 10, 2012 at 6:44 PM, Ricardo Moran richi.moran@gmail.com wrote: Sorry, the last change set had a bug when the slot was removed while the dialog was still open. I fixed it in this new change set.
Cheers, Richo
On Fri, Feb 10, 2012 at 6:41 PM, Ricardo Moran richi.moran@gmail.com wrote: Hi,
I've made a partial update on the new dialog (change set attached). I haven't added the type-specific "decimal places" yet, but I replaced the menu items "rename <var>" and "change value type" with the item "change variable info...".
I also fixed the code a little, removed the title, and added a couple of #translated sends.
Cheers, Richo
On Thu, Feb 9, 2012 at 9:56 PM, Scott Wallace scott.wallace@squeakland.org wrote: On Feb 9, 2012, at 9:10 AM, Steve Thomas wrote:
Ed Team should think about Quick Guides on Variables, and perhaps balloon help when you mouse over the variable types.
Right -- and it would be very good if in the gallery or guides or *somewhere* we had examples illustrating use of every one of the types.
-- Sott _______________________________________________ etoys-dev mailing list etoys-dev@squeakland.org http://lists.squeakland.org/mailman/listinfo/etoys-dev
<NewVariable2.4.cs>