I found, the bug has to do something with calculating of row position in
ui/table/pane/Scroller.js in function _getRowForPagePos.
There is used the paneClipperTopY to calculate tableY

        var tableY = scrollY + pageY - paneClipperTopY;
        var row = Math.floor(tableY / rowHeight);

But paneClipperTopY becomes negativ if the Top of the table scrolls out of
visible browser, so the result is incorrect...
May be this could help to fix the bug?



westor wrote:
> 
> Hallo Hugh,
> thank you for spending your time with me ;-)
> Here is a snippet for testing, as required. Please use firefox and scroll
> the browser and see what happens, if you click a line.
> 
> // in main
> this.base(arguments);
> var d = qx.ui.core.ClientDocument.getInstance();
> var inlineWidget = new qx.ui.basic.Inline("qxframe");
> inlineWidget.setHeight("100%");
> inlineWidget.setWidth("auto");
> var vbx = new qx.ui.layout.VerticalBoxLayout();
> vbx.setWidth("auto");
> vbx.set(
> {
>       left    : 5,
>       top     : 5,
>       bottom  : 30,
>       spacing : 5
> });
> var tableModel = new qx.ui.table.model.Simple();
> tableModel.setColumns([ "1", "2", "3" ]);
> var table = new qx.ui.table.Table(tableModel);
> with (table)
> {
>       set(
>       {
>               width  : "100%",
>               height : "1*",
>               border : "inset-thin",
>               rowHeight : 63
>       });
> 
> getSelectionModel().setSelectionMode(qx.ui.table.selection.Model.SINGLE_SELECTION);
> }
> var rowData = new Array();
> for (var row = 0; row < 1000; row++) {
>   rowData.push([ "xxxx", "xxxx", "xxxx" ]);
> }
> tableModel.setData(rowData);
> vbx.add(table);
> inlineWidget.add(vbx);
> d.add(inlineWidget);
> 
> // in html file
> <table height="2000"><tr><td>
> <div id="qxframe" style="hight:100%"><br>
>  resource/indicator_waitanim.gif  Laden.... bitte warten
> </div>
> </td></tr></table>
> 
> 
> 
> 
> Hugh Gibson wrote:
>> 
>>> Firefox:
>>> If you scroll the browser (not the table widget), the selected row 
>>> is not the row where the mouse pointer is. Instad a row is selected,
>>> which is [scrolled pixel] below mouse pointer.
>>> This works well in IE - at least in IE6. (Please could someone give
>>> feedback, if the application runs in IE7?)
>> 
>> Works fine in IE7. 
>> 
>>> Internet Explorer 6:
>>> - A double click in a table row opens a qx.ui.window.Window with an 
>>> Iframe inside. But the underlaying browser window has an advertising
>>> box in the right, and inside is a combobox ("Geschlecht"). This
>>> combobox is not covered by the qooxdoo window.
>> 
>> OK in IE7.
>> 
>>> - If you select a combobox (Kategorie or Unterkategorie), the popup 
>>> of the combo appears for a very shot time in the right of browser
>>> window. This lets the application display flicker... not nice, but
>>> not really terrible.
>>> Also something like this happens in firefox too, but it's outside 
>>> the visible area. Only the horizontal scollbar appears for a short
>>> time.
>> 
>> Same here. Might be due to this code in qx.ui.popup.Popup._beforeAppear:
>> 
>>       if (this.getRestrictToPageOnOpen())
>>       {
>>         this._wantedLeft = this.getLeft();
>> 
>>         if (this._wantedLeft != null)
>>         {
>> // Move the popup out of the view so its size could be
>> calculated before
>>           // it is positioned.
>>           this.setLeft(10000);
>> 
>>           if (this.getElement() != null)
>>           {
>>             // The popup was already visible once before
>>             // -> Move it immediately before it gets visible again
>>             this.getElement().style.left = 10000;
>>           }
>>         }
>>       }
>> 
>> Might be worth trying to set restrictToPageOnOpen false for the popup
>> window in the combo box.
>> 
>> Regarding the other problems, the best way to get help is to build a
>> sample application that shows the problem. Then it's easier to debug. It
>> might even help if you did a release build with linefeeds in it - then
>> some work could be done on tracking down the issues with your app online.
>> 
>> Hugh
>> 
>> -------------------------------------------------------------------------
>> This SF.net email is sponsored by: Splunk Inc.
>> Still grepping through log files to find problems?  Stop.
>> Now Search log events and configuration files using AJAX and a browser.
>> Download your FREE copy of Splunk now >>  http://get.splunk.com/
>> _______________________________________________
>> qooxdoo-devel mailing list
>> qooxdoo-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
>> 
>> 
> 
> 

-- 
View this message in context: 
http://www.nabble.com/table-row-select-bug-%28FF%29-and-other-%28smaller%29-probs-with-online-app-tf4391216.html#a12553823
Sent from the qooxdoo-devel mailing list archive at Nabble.com.


-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
_______________________________________________
qooxdoo-devel mailing list
qooxdoo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

Reply via email to