Tom,

 I checked out the header_footer branch just now and it still had a
problem.  I am not familiar with how to submit patches etc but adding the
following code to Grid.java -->updateSelection(): fixes it for me.

    private Event updateSelection( GridItem item, int stateMask ) {
.... Before .....
            boolean shift = false;
            boolean ctrl = false;

            if (( stateMask & SWT.SHIFT ) == SWT.SHIFT) {
                shift = true;
            }

            if (( stateMask & SWT.CTRL ) == SWT.CTRL) {
                ctrl = true;
            }
......................
... After ....
            boolean shift = false;
            boolean ctrl = false;
            boolean cmd = true;

            if (( stateMask & SWT.SHIFT ) == SWT.SHIFT) {
                shift = true;
            }

            if (( stateMask & SWT.CTRL ) == SWT.CTRL) {
                ctrl = true;
            }

            if (( stateMask & SWT.COMMAND ) == SWT.COMMAND) {
                if (SWT.getPlatform().equals( "carbon" )) {
                    ctrl = cmd = true;
                }
            }
.......................


Thanks,
~jake

On Fri, Nov 21, 2008 at 5:56 PM, Tom Schindl <[EMAIL PROTECTED]> wrote:

> Fixed in header_footer branch of grid.
>
> Tom
>
> Jake Fisher schrieb:
> > So I am using the last built version on grid and gridviewer on the
> > nebula site and I am not sure how different that is from the current
> > trunk, so this may or may not still be an issue.
> >
> > In OSX I want to use cmd->click to select multiple rows particularly
> > since ctrl->click activates the context menu. The current Grid code
> > available for download does not support that and I wanted to add it as a
> > bug if it was still a problem.
> >
> > Thanks,
> > ~jake
> >
> > PS: Also well going trhough the code I noticed a couple checks for SWT
> > carbon to pain things differently come 3.5 there with swt cocoa I am not
> > sure what this will effect or not effect.
> >
> >
> > ------------------------------------------------------------------------
> >
> > _______________________________________________
> > nebula-dev mailing list
> > [email protected]
> > https://dev.eclipse.org/mailman/listinfo/nebula-dev
>
> _______________________________________________
> nebula-dev mailing list
> [email protected]
> https://dev.eclipse.org/mailman/listinfo/nebula-dev
>
_______________________________________________
nebula-dev mailing list
[email protected]
https://dev.eclipse.org/mailman/listinfo/nebula-dev

Reply via email to