ok, checked in after modifying per Pablo's comment. I made it so the tooltip doesn't automagically set its parent to be clickable and updated the doc to say that the parent needs to be clickable. (sorry, it actually happened in two check ins... still getting used to the review/commit scripts)

On Sun, Mar 18, 2007 at  6:12 PM, Pablo Kang wrote:

Not sure if we want to implicitly make the parent clickable. A view may be disabled and not want to be clickable. Maybe have the parent control its clickability to pass mouse events to the tooltip. I like having tooltip extend node.

On Sun, 18 Mar 2007, Sarah Allen wrote:

Ok, here's the other part of that change. Oddly, the UTF-8 problem didn't appear. After I check this in, I'll see if there's some rogue file that gets included when I make a changeset from the top of the trunk directory.

Not sure who is a good reviewer for incubator components, but anyone outght to be able to review this. I even updated the test :)

Change 20070318-sallen-4 by [EMAIL PROTECTED] on 2007-03-18 15:51:12 PDT in /cygdrive/c/laszlo/svn/openlaszlo/trunk/lps/components/incubator for http://svn.openlaszlo.org/openlaszlo/trunk/lps/components/incubator
Summary: tooltip should extend node not view

Bugs Fixed: LPP-3715 tooltip should extend node not view

Technical Reviewer: (pending)
QA Reviewer: (pending)
Doc Reviewer: (pending)

Documentation:

Release Notes:

Details:
This change makes it so you can attach a tooltip to a view with a resource
and not get errors telling you that a view with a resource can't
have subviews.|  There's a single tooltipvew that is manipulated by
the tooltip class -- so tooltip doesn't actually need to be a view.

Files:
M      tooltip/tooltip.lzx
M      test/tooltip-test.lzx

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20070318-sallen-4.tar


Reply via email to