Thanks for the tip. Didn't help exactly because the it was on a different event (the mousedown, not the click) and on a different object (the grid's view's mainBody, whatever that is...)
However I made the fix here: http://github.com/GeoNode/geonode/commit/6b54ac1f5ebdf0f863a565107ed1da45d309aa12 Taking another crack at it, I found a way around this by adding an 'init' event to the RowExpander. I guess it's cleaner, though really it feels like hacking around the RowExpander in a more convoluted way. http://github.com/GeoNode/geonode/commit/3fb9f92e1d10c1797913d77fe359a6f5020f1c18 Oh well. Will let this one rest I think unless anybody has any other objections/suggestions. On Tue, Jul 6, 2010 at 6:56 PM, Andreas Hocevar <[email protected]>wrote: > 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. > > -- Sebastian Benthall OpenGeo - http://opengeo.org
