Hey Seb- you can try to not override onMouseDown at all, but instead add the following as first line of your rowclick handler:
evt.stopEvent(); This is a good idea in general, to avoid other event handlers being triggered (like e.g. field editors). -Andreas. On Jul 6, 2010, at 23:07 , Sebastian Benthall wrote: > Emailing list for group edification, but mostly for Andreas: > > http://github.com/GeoNode/geonode/commit/55c0b999248a273fa3179e7e64e2c021b9dfb411 > > Mad this change to the Ext-based MapGrid. I note it here because I couldn't > figure out a cleaner (event-driven) way to disable the expander plugin button > except the hacky way I did it. > > I think the way the RowExpander is designed makes it much more trouble than > its worth to unbind the 'mousedown' event (the binding of the rowToggle > behavior to the mousedown event happens when the grid's view is rendered, but > the declaration happens when the expander itself is initialized, so the > timing is tricky--i tried!). But I wanted to invite your review in case you > see a better way to do it that escaped me. > > -- > Sebastian Benthall > OpenGeo - http://opengeo.org > -- Andreas Hocevar OpenGeo - http://opengeo.org/ Expert service straight from the developers.
