it seems very expensive to me.
 
the .scrollTop = is a dom operation i'm guessing ?


________________________________

        From: Dietrich Streifert [mailto:[EMAIL PROTECTED] 
        Sent: Monday, August 06, 2007 12:07 PM
        To: qooxdoo Development
        Subject: Re: [qooxdoo-devel] qx.ui.table.Table with (too) many columns
        
        
        Ah ok my mistake. I'm using some revision of branches/legacy_0_7_x. 
        
        Ok then this makes sense.
        
        Bart van der Werf schrieb: 

                on my version of qx widget.js: 6878 is 
                 
                setScrollTop : function(nScrollTop)

                {

                this._visualPropertyCheck();

                this._getTargetNode().scrollTop = nScrollTop;

                },

                 

                 


________________________________

                        From: Dietrich Streifert [mailto:[EMAIL PROTECTED] 
                        Sent: Monday, August 06, 2007 11:42 AM
                        To: qooxdoo Development
                        Subject: Re: [qooxdoo-devel] qx.ui.table.Table with 
(too) many columns
                        
                        
                        The profile indicates that a huge amount of time is 
spent in Widget.js at line 6878:
                        
                            setScrollLeft : function(nScrollLeft)
                            {
                              this._visualPropertyCheck();
                              this._getTargetNode().scrollLeft = nScrollLeft;
                            },
                        
                        So this might be worse investigating why this is done. 
Scroll left means horizontal scrolling.
                        
                        Did you perform any horizontal scrolling during your 
profile catch?
                        
                        
                        Bart van der Werf schrieb: 

                                Sorry i need to redo the baseline, i see that 
the original baseline was on twenty columns, this one on 7 columns


________________________________

                                        From: Bart van der Werf [mailto:[EMAIL 
PROTECTED] 
                                        Sent: Monday, August 06, 2007 10:36 AM
                                        To: qooxdoo Development
                                        Subject: Re: [qooxdoo-devel] 
qx.ui.table.Table with (too) many columns
                                        
                                        
                                        ok, i renamed it to 

                                        
webgui.ui.comtecgrid.ComtecGridDataCellRenderer

                                        so it might appear as a different name 
in the profile

                                        it does feel abit faster.

                                         

                                        firefox crashed on the first attempt to 
profile :)

                                         

                                        on a grid with 8 columns and a few 
thousand rows.

                                         

# full-url

file-name

function-name

start-line

end-line

call-count

recurse-depth

total-time

min-time

max-time

avg-time

own-total-time

own-min-time

own-max-time

own-avg-time

http://localhost:4499/Website/qooxdoo-0,7,1-sdk/frontend/framework/source/class/qx/ui/core/Widget,js

Widget,js

anonymous

6878

6881

1036

0

9859,6

0,0

46,9

9,5

9859,6

0,0

46,9

9,5

http://localhost:4499/Website/qooxdoo-0,7,1-sdk/frontend/framework/source/class/qx/ui/table/pane/Pane,js

Pane,js

anonymous

491

612

700

0

24844,2

0,0

78,1

35,5

7984,5

0,0

62,5

11,4

http://localhost:4499/Website/prototype/source/class/webgui/ui/comtecgrid/ComtecGridDataCellRenderer,js

ComtecGridDataCellRenderer,js

anonymous

88

95

151656

0

7234,5

0,0

15,6

0,1

7234,5

0,0

15,6

0,1

http://localhost:4499/Website/qooxdoo-0,7,1-sdk/frontend/framework/source/class/qx/ui/table/rowrenderer/Default,js

Default,js

anonymous

447

482

19593

0

4984,5

0,0

15,6

0,3

4968,9

0,0

15,6

0,3

http://localhost:4499/Website/prototype/source/class/webgui/ui/comtecgrid/ComtecGridTableModel,js

ComtecGridTableModel,js

anonymous

61

82

151656

0

2343,8

0,0

15,6

0,0

1734,4

0,0

15,6

0,0

http://localhost:4499/Website/qooxdoo-0,7,1-sdk/frontend/framework/source/class/qx/ui/core/Widget,js

Widget,js

anonymous

6930

6933

702

0

937,5

0,0

31,3

1,3

937,5

0,0

31,3

1,3

http://localhost:4499/Website/qooxdoo-0,7,1-sdk/frontend/framework/source/class/qx/ui/table/pane/Model,js

Model,js

anonymous

210

213

155287

0

1359,4

0,0

15,6

0,0

656,3

0,0

15,6

0,0

http://localhost:4499/Website/qooxdoo-0,7,1-sdk/frontend/framework/source/class/qx/core/Target,js

Target,js

anonymous

276

312

31455

15

14640,9

0,0

62,5

0,5

468,8

0,0

15,6

0,0

http://localhost:4499/Website/gossip/Gossip,js

Gossip,js

anonymous

493

495

171147

0

375,0

0,0

15,6

0,0

375,0

0,0

15,6

0,0

http://localhost:4499/Website/qooxdoo-0,7,1-sdk/frontend/framework/source/class/qx/core/Object,js

Object,js

anonymous

543

567

13340

0

375,0

0,0

15,6

0,0

375,0

0,0

15,6

0,0

                                        average from 240 to 78 on Pane !! very 
good

                                         

                                         


________________________________

                                        From: Dietrich Streifert [mailto:[EMAIL 
PROTECTED] 
                                        Sent: Monday, August 06, 2007 10:08 AM
                                        To: qooxdoo Development
                                        Subject: Re: [qooxdoo-devel] 
qx.ui.table.Table with (too) many columns
                                        
                                        
                                        Can you do the same pofile using 
                                        
                                           qx.ui.table.pane.Pane.USE_ARRAY_JOIN 
= false;
                                        
                                        and using the attached direct 
cellrenderer please?
                                        
                                        You have to change the extension from 
.j_ to .js and you have to adapt the namespace to your needs. Give it a try.
                                        
                                        
                                        Bart van der Werf schrieb: 

                                         
                                        From: Dietrich Streifert
                                                
                                          

                                        Does the profile include the initial 
showup of the table?
                                            

                                                
                                        This profile is of scrolling one row up 
and down for a while.
                                        
                                        
                                        
-------------------------------------------------------------------------
                                        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
                                        [email protected]
                                        
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
                                          


                                        -- 
                                        Mit freundlichen Grüßen
                                        Dietrich Streifert
                                        --
                                        Visionet GmbH
                                        Firmensitz: Am Weichselgarten 7, 91058 
Erlangen
                                        Registergericht: Handelsregister Fürth, 
HRB 6573
                                        Geschäftsführer: Stefan Lindner
                                        
                                        
                                              

                                
________________________________


                                
-------------------------------------------------------------------------
                                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
                                [email protected]
                                
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
                                  


                        -- 
                        Mit freundlichen Grüßen
                        Dietrich Streifert
                        --
                        Visionet GmbH
                        Firmensitz: Am Weichselgarten 7, 91058 Erlangen
                        Registergericht: Handelsregister Fürth, HRB 6573
                        Geschäftsführer: Stefan Lindner
                        
                        
                            

                
________________________________


                
-------------------------------------------------------------------------
                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
                [email protected]
                https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
                  


        -- 
        Mit freundlichen Grüßen
        Dietrich Streifert
        --
        Visionet GmbH
        Firmensitz: Am Weichselgarten 7, 91058 Erlangen
        Registergericht: Handelsregister Fürth, HRB 6573
        Geschäftsführer: Stefan Lindner
        
        

-------------------------------------------------------------------------
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
[email protected]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

Reply via email to