Actually, had to remove the stopEvent() since it was breaking the links on the expander and the grid elements.
Could just stop the event propagation and not the browser default behavior, but I'm not sure if that really gets us anything? On Wed, Jul 7, 2010 at 11:26 AM, Sebastian Benthall <[email protected]> wrote: > 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 > > -- Sebastian Benthall OpenGeo - http://opengeo.org
