No, it's not just the debugger, this change seems like it causes any selectable text view to not be selectable. <br><br><br><br><br><div class="gmail_quote">On Fri, Aug 28, 2009 at 4:20 PM, P T Withington <span dir="ltr"><<a href="mailto:ptw@laszlosystems.com">ptw@laszlosystems.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">So, does the clickability of the debugger text take some completely different path than the clickability of normal views? Maybe we are barking up the wrong tree here.<div>
<div></div><div class="h5"><br>
<br>
On 2009-08-28, at 15:51EDT, Henry Minsky wrote:<br>
<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Yeah, I mean to look the clickdiv. Max's changes only affect the<br>
root canvas sprite clickdiv width and height (the code is all in "if<br>
(isroot) { ...}" blocks) .<br>
<br>
I was thinking maybe the "100%"<br>
was confusing the CSS,but I tried a canvas with an absolute width and<br>
height, and it<br>
still doesn't work.<br>
<br>
What is weird is when I comment out max's change (which makes text be<br>
selectable again) ) and ask for the canvas clickdiv<br>
width and height, they have empty string values.<br>
<br>
So I am missing something here. I thought it was going to be that the canvas<br>
clickdiv had zero size, but that doesn't seem to be the problem.<br>
<br>
lzx> canvas.sprite.__LZclickcontainerdiv.style.width<br>
''<br>
lzx> canvas.sprite.__LZclickcontainerdiv.style.height<br>
''<br>
lzx> canvas.sprite.__LZclickcontainerdiv<br>
«HTMLDivElement#0| This is the<br>
canvas/@sprite/@__LZclickcontainerdiv/div.lzcanvasclickdiv[@style="clip:<br>
rect(0px, 1357px, 535px, 0px); display: none;"]»<br>
lzx><br>
<br>
<br>
<br>
On Fri, Aug 28, 2009 at 3:43 PM, P T Withington <<a href="mailto:ptw@laszlosystems.com" target="_blank">ptw@laszlosystems.com</a>>wrote:<br>
<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Isn't it the clickdiv you need to look at, not the context div? But, if<br>
the clickdiv is also 0-sized, that would explain why there is nothing to<br>
click on.<br>
<br>
This is the clickdiv for the debugger content view:<br>
<br>
<div id="click/LzDebugWindow/@middle/@content/text"<br>
class="lztextcontainer_click" style="z-index: 2; clip: rect(0px, 4096px,<br>
66px, 0px); left: 0px;"/><br>
<br>
<br>
It's clip is the right size, but it itself has no height or width. I<br>
wonder if it has something to do with:<br>
<br>
this.quirks.size_blank_to_zero<br>
<br>
you could try setting that to false and see.<br>
<br>
<br>
On 2009-08-28, at 15:08EDT, Henry Minsky wrote:<br>
<br>
In the sprite constructor, I see a value of zero being assigned to width<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
and<br>
height here<br>
<br>
if (quirks.fix_contextmenu) {<br>
var cxdiv = document.createElement('div');<br>
cxdiv.className = 'lzcanvascontextdiv';<br>
<a href="http://cxdiv.id" target="_blank">cxdiv.id</a> = 'lzcanvascontextdiv';<br>
root.appendChild(cxdiv);<br>
cxdiv.owner = this;<br>
cxdiv.style.width = this._w;<br>
cxdiv.style.height = this._h;<br>
Debug.info("setting cvdiv.style.width", this._w);<br>
Debug.info("setting cvdiv.style.height", this._h);<br>
this.__LZcontextcontainerdiv = cxdiv;<br>
}<br>
<br>
That prints out "0pt" for the values when it runs...<br>
<br>
On Fri, Aug 28, 2009 at 2:28 PM, Henry Minsky <<a href="mailto:henry.minsky@gmail.com" target="_blank">henry.minsky@gmail.com</a><br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
wrote:<br>
</blockquote>
<br>
This regression came with changeset<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
r14554 | max | 2009-08-24 21:05:44 -0400 (Mon, 24 Aug 2009) | 18 lines<br>
<br>
Change 20090821-maxcarlson-Y by maxcarlson@Bank on 2009-08-21 17:33:09<br>
PDT<br>
in /Users/maxcarlson/openlaszlo/trunk-clean<br>
for <a href="http://svn.openlaszlo.org/openlaszlo/trunk" target="_blank">http://svn.openlaszlo.org/openlaszlo/trunk</a><br>
<br>
Summary: Eliminate scrollbars for oversized divs<br>
<br>
<br>
<br>
<br>
<br>
On Fri, Aug 28, 2009 at 1:44 PM, Henry Minsky <<a href="mailto:henry.minsky@gmail.com" target="_blank">henry.minsky@gmail.com</a><br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
wrote:<br>
</blockquote>
<br>
I'll try and locate where this regressed<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
<br>
On Fri, Aug 28, 2009 at 1:01 PM, Max Carlson <<a href="mailto:max@laszlosystems.com" target="_blank">max@laszlosystems.com</a><br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
wrote:<br>
</blockquote>
<br>
I swear this was working the other day. What does a binary search say?<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
-----Original Message-----<br>
From: Henry Minsky <<a href="mailto:henry.minsky@gmail.com" target="_blank">henry.minsky@gmail.com</a>><br>
Sent: Friday, August 28, 2009 8:58 AM<br>
To: <a href="mailto:andre.bargull@udo.edu" target="_blank">andre.bargull@udo.edu</a>; P. Tucker Withington <<a href="mailto:ptw@openlaslzo.org" target="_blank">ptw@openlaslzo.org</a>><br>
Cc: <a href="mailto:max@laszlosystems.com" target="_blank">max@laszlosystems.com</a>; <a href="mailto:laszlo-reviews@openlaszlo.org" target="_blank">laszlo-reviews@openlaszlo.org</a><br>
Subject: Re: For Review: Change 20090827-hqm-s Summary: make context<br>
menus 'hide' behavior match swf8<br>
<br>
Hey Tucker, do you happen to know if something changed recently with<br>
text<br>
or selection that makes it not possible to select debugger messages in<br>
the<br>
DHTML debug window?<br>
<br>
<br>
<br>
And I'm currently unable to inspect messages in the DHTML debugger (at<br>
least under Firefox3, trunk@14603), is that a known issue?<br>
<br>
<br>
<br>
On 8/28/2009 5:39 AM, Henry Minsky wrote:<br>
Change 20090827-hqm-s by hqm@badtzmaru.home on 2009-08-27 23:33:23 EDT<br>
in /Users/hqm/openlaszlo/trunk-diamond/WEB-INF/lps/lfc<br>
for <a href="http://svn.openlaszlo.org/openlaszlo/trunk/WEB-INF/lps/lfc" target="_blank">http://svn.openlaszlo.org/openlaszlo/trunk/WEB-INF/lps/lfc</a><br>
<br>
Summary: make context menus 'hide' behavior match swf8<br>
<br>
New Features:<br>
<br>
Bugs Fixed: LPP-8218<br>
<br>
Technical Reviewer: max<br>
QA Reviewer: andre<br>
Doc Reviewer: (pending)<br>
<br>
Documentation:<br>
<br>
Release Notes:<br>
<br>
Details:<br>
<br>
+ Change to make a context menu hide if you mouse up or down outside of<br>
it<br>
<br>
<br>
</blockquote>
<br>
<br>
--<br>
Henry Minsky<br>
Software Architect<br>
<a href="mailto:hminsky@laszlosystems.com" target="_blank">hminsky@laszlosystems.com</a><br>
<br>
<br>
<br>
<br>
</blockquote>
<br>
--<br>
Henry Minsky<br>
Software Architect<br>
<a href="mailto:hminsky@laszlosystems.com" target="_blank">hminsky@laszlosystems.com</a><br>
<br>
<br>
<br>
<br>
</blockquote>
<br>
--<br>
Henry Minsky<br>
Software Architect<br>
<a href="mailto:hminsky@laszlosystems.com" target="_blank">hminsky@laszlosystems.com</a><br>
<br>
</blockquote>
<br>
<br>
</blockquote>
<br>
<br>
-- <br>
Henry Minsky<br>
Software Architect<br>
<a href="mailto:hminsky@laszlosystems.com" target="_blank">hminsky@laszlosystems.com</a><br>
</blockquote>
<br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Henry Minsky<br>Software Architect<br><a href="mailto:hminsky@laszlosystems.com">hminsky@laszlosystems.com</a><br><br><br>