Mike, your fix looks good :).. the whole class was a mix of tabs/spaces. just reformatted & committed it.
..ede -------- Forwarded Message -------- Subject: Re: [JPP-Devel] SVN: [5270] core/trunk Date: Mon, 19 Dec 2016 10:48:31 +0100 From: [email protected] To: [email protected] hey Mike, yeah had that still in my inbox, like so much else ;(.. will work through it over the holidays.. btw. indention of your additions looks wrong (tabs?).. ede On 19.12.2016 10:37, [email protected] wrote: > Revision: 5270 > http://sourceforge.net/p/jump-pilot/code/5270 > Author: michaudm > Date: 2016-12-19 09:37:34 +0000 (Mon, 19 Dec 2016) > Log Message: > ----------- > fix #440 about shift-selection problem between layerView and attributePanel > > Modified Paths: > -------------- > core/trunk/ChangeLog > > core/trunk/src/com/vividsolutions/jump/workbench/ui/cursortool/AbstractCursorTool.java > > Modified: core/trunk/ChangeLog > =================================================================== > --- core/trunk/ChangeLog 2016-12-19 06:50:24 UTC (rev 5269) > +++ core/trunk/ChangeLog 2016-12-19 09:37:34 UTC (rev 5270) > @@ -3,6 +3,9 @@ > # 2. make sure that lines break at 80 chars for constricted display > situations > #<-------------------------------- 80 chars > ----------------------------------># > > +2016-12-19 mmichaud <[email protected]> > + * fix #440 about shift-selection problem between layerView and > attributePanel > + > 2016-12-17 ede > * upgrade commons-lang to version 3.5 > > > Modified: > core/trunk/src/com/vividsolutions/jump/workbench/ui/cursortool/AbstractCursorTool.java > =================================================================== > --- > core/trunk/src/com/vividsolutions/jump/workbench/ui/cursortool/AbstractCursorTool.java > 2016-12-19 06:50:24 UTC (rev 5269) > +++ > core/trunk/src/com/vividsolutions/jump/workbench/ui/cursortool/AbstractCursorTool.java > 2016-12-19 09:37:34 UTC (rev 5270) > @@ -709,6 +709,10 @@ > } > > public void keyPressed(KeyEvent e) { > + // Save modifiers even if the componentWithFocus is not the > LayerView > + // otherwise, when entering or exiting another component, one > of the > + // modification (pressed or released) is not saved > + saveModifiers(e); > if(!componentWithFocusIsHandledByCursorTools()) > return; > > @@ -719,10 +723,13 @@ > // System.out.println("snap off"); > > showMsg("com.vividsolutions.jump.workbench.ui.cursortool.AbstractCursorTool.snapping-off"); > } > - saveModifiers(e); > } > > public void keyReleased(KeyEvent e) { > + // Save modifiers even if the componentWithFocus is not the > LayerView > + // otherwise, when entering or exiting another component, one > of the > + // modification (pressed or released) is not saved > + saveModifiers(e); > if(!componentWithFocusIsHandledByCursorTools()) > return; > > @@ -733,7 +740,6 @@ > // System.out.println("snap on"); > > showMsg("com.vividsolutions.jump.workbench.ui.cursortool.AbstractCursorTool.snapping-on"); > } > - saveModifiers(e); > } > > private void saveModifiers(KeyEvent e){ > > > ------------------------------------------------------------------------------ > Check out the vibrant tech community on one of the world's most > engaging tech sites, SlashDot.org! http://sdm.link/slashdot > _______________________________________________ > Jump-pilot-devel mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel > ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, SlashDot.org! http://sdm.link/slashdot _______________________________________________ Jump-pilot-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
