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

Reply via email to