Hi Larry, Jukka,

Thanks for the details,
I could find the bug report in the bug database
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7122141

I agree that having the check symbol visible is more important than the icon,
so I will apply Larry's patch.
I liked the icons too and voted for a fix in the Oracle's bug database as it had a very low priority.

Michaël

Le 30/03/2012 15:13, Rahkonen Jukka a écrit :
Hi,
Checks does not show for me on Vista 32-bit either with the default skin. With most other skins the checkbox is visible.
-Jukka Rahkonen-

    ------------------------------------------------------------------------
    Larry Becker  wrote:
    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
    <mailto: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 list
        Jump-pilot-devel@lists.sourceforge.net  
<mailto:Jump-pilot-devel@lists.sourceforge.net>
        https://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
        <mailto:Jump-pilot-devel@lists.sourceforge.net>
        https://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

------------------------------------------------------------------------------
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