Hi Michaël, I've attached screen captures to illustrate the problem. Notice that XP displays both icons and checks, but Windows 7 shows only icons.
Now that I look closely, I see that the icons themselves provide a subtle feedback that the item is selected. I prefer the actual checks myself. Larry 2012/3/29 Michaël Michaud <michael.mich...@free.fr> > Hi Larry, > > Thanks for the tip. > Can you give more details about the configuration that does not work, > I can't reproduce the problem : > Windows Vista SP2 64bits + jre6 32 bits works as it should > Windows Vista SP2 64bits + jre7 32 bits works as it should > Windows Vista SP2 64bits + jre7 64 bits works as it should. > > Michaël > > Le 29/03/2012 21:55, Larry Becker a écrit : > > Hi, > > I noticed that the Selectable and Editable menu items won't check on > Vista and Win7 due to an interface change. If we get rid of the menu > icons, which don't really add much, the functionality is restored. I have > included a patch to try out the fix. > > regards, > > Larry > > Index: > C:/eclipse3/workspace/OpenJumpSvn/src/com/vividsolutions/jump/workbench/JUMPConfiguration.java > =================================================================== > --- > C:/eclipse3/workspace/OpenJumpSvn/src/com/vividsolutions/jump/workbench/JUMPConfiguration.java > (revision 2751) > +++ > C:/eclipse3/workspace/OpenJumpSvn/src/com/vividsolutions/jump/workbench/JUMPConfiguration.java > (working copy) > @@ -507,10 +507,10 @@ > .getFrame().getLayerNamePopupMenu(); > > featureInstaller.addPopupMenuItem(layerNamePopupMenu, > editablePlugIn, > - editablePlugIn.getName(), true, EditablePlugIn.ICON, > editablePlugIn > + editablePlugIn.getName(), true, null, editablePlugIn > .createEnableCheck(workbenchContext)); > featureInstaller.addPopupMenuItem(layerNamePopupMenu, > selectablePlugIn, > - selectablePlugIn.getName(), true, selectablePlugIn.ICON, > selectablePlugIn > + selectablePlugIn.getName(), true, null, selectablePlugIn > .createEnableCheck(workbenchContext)); > layerNamePopupMenu.addSeparator(); // =================== > > > > > ------------------------------------------------------------------------------ > This SF email is sponsosred by: > Try Windows Azure free for 90 days Click Here > http://p.sf.net/sfu/sfd2d-msazure > > > > _______________________________________________ > Jump-pilot-devel mailing > listJump-pilot-devel@lists.sourceforge.nethttps://lists.sourceforge.net/lists/listinfo/jump-pilot-devel > > > > > ------------------------------------------------------------------------------ > This SF email is sponsosred by: > Try Windows Azure free for 90 days Click Here > http://p.sf.net/sfu/sfd2d-msazure > _______________________________________________ > Jump-pilot-devel mailing list > Jump-pilot-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel > >
<<attachment: XP.PNG>>
<<attachment: Win7.png>>
------------------------------------------------------------------------------ This SF email is sponsosred by: Try Windows Azure free for 90 days Click Here http://p.sf.net/sfu/sfd2d-msazure
_______________________________________________ Jump-pilot-devel mailing list Jump-pilot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel