Sluggish mouseover effect tree/grid

[Animal]

I'll talk to Don some more about this. The 20ms setting feels about right. Have you tried it?

[mankz]

20 is decent, still doesn't feel like Chrome (any delay is noticeable).

Btw, setting it to 0 produces an error...

[Animal]

That's a bug for sure. I'll create a ticket.

Fix:

Ext.override(Ext.view.View, {
    afterRender: function(){
        var me = this,
            onMouseOverOut = me.mouseOverOutBuffer ? me.onMouseOverOut : me.handleMouseOverOrOut;

        me.callParent();
        me.mon(me.getTargetEl(), {
            scope: me,
            /*
             * We need to make copies of this since some of the events fired here will end up triggering
             * a new event to be called and the shared event object will be mutated. In future we should
             * investigate if there are any issues with creating a new event object for each event that
             * is fired.
             */
            freezeEvent: true,
            click: me.handleEvent,
            mousedown: me.handleEvent,
            mouseup: me.handleEvent,
            dblclick: me.handleEvent,
            contextmenu: me.handleEvent,
            keydown: me.handleEvent,
            mouseover: onMouseOverOut,
            mouseout:  onMouseOverOut
        });
    }
});

[dongryphon]

Reducing the buffer to 20ms seems to feel pretty good - the problem with no delay is that it actually creates a real CPU hit to try to keep up with every mousemove. The delay was added because users on (cough) IE (cough) reported the CPU spike. We'll see how the new delay feels.

Of course you (or someone else reading this) can override this in your app

Ext.define('MyApp.patches.view.View', {
    target: 'Ext.view.View',

    mouseOverOutBuffer: 0
});

Thanks for reporting it - let us know how the smaller buffer value feels?

[dongryphon]

I see my window was stale when I posted my response :P

I guess we just need an Ext.support.EyeCandy feature detector

[jchau]

Well that's possible. But trackMouseOver adds a class and fires events which form part of an API which is in wide use.

If all you need in your app is a hover state, then you could use

    trackMouseOver: false

and add

.x-grid-row:hover .x-grid-cell {
    background-color: whatever;
}

This doesn't work for locking grids since there's two separate grids...

[aconran]

The :hover approach will make IE7 truly suffer.