Re: [Jajuk-dev] Jajuk OS X right click issue

[Bertrand F. <ber...@fl...>]

Hi Daniel,

(If you don't mind, I CC the jajuk list as it may generate feedback)

d.h...@db... wrote:
> Hi Bertrand,
>
> Am 28.07.2007 um 16:23 schrieb Bertrand Florat:
>
>>>
>>> Now everything works fine. Right click behavior is as expected. Except
>>> for the area where the sources are configured.
>> Thanks Daniel for the feedback, which version did you finally used (it's
>> because I'd like to now if JNLP distribution finally work under OSX) ?
>
> Well, the web start link you sent me then worked. At the moment
> neither link works.
>
> Exception is java.net.ConnectException: Operation timed out

Strange... works for me. Anyway, here's the latest build:
http://repository.jajuk.info/jajuk/testing/build-1.3/jajuk-java-installer-1.3.11.jar
If you can, could you please confirm it fixes your problem in the device
view ? (BTW, please check there is no regression in trees and tables as
I generalized my last fix everywhere)

>
>>
>>>
>>> If I CTRL+click there on an existing source straight after the popup
>>> menu shows up the source configuration dialog opens  and hides the
>>> menu. So you can't use the context menu there. Maybe you should check
>>> the modifiers in the mouse event and not show up the config dialog if
>>> a modifier is used.
>>
>> It remind something, I made  a fix, could you please check it at
>> http://florat.net/jajuk-java-installer-1.3.11.jar ?
>> The problem, AFAIK is that sometimes, for unknown reason, right click is
>> not recognized as a popupTrigger (I reproduced it under Linux) so I have
>> to add this test on the mouseClicked() method:
>>
>> public void mouseClicked(MouseEvent e) {
>>         // Make sure not to handle events for popup handling, keep the
>>         //test on button 3
>>         //for unknown reason, a right click can be seen as non
>> popupTrigger
>>         if (e.isPopupTrigger() || e.getButton() == 3) {
>>             return;
>>         }
>>
>> any idea on this ?
>
> Hmm, reminds me of an article about mouse listeners where it said that
> mouse listeners and in particular the popup menu trigger is highly
> platform dependent. I normally only use the mousePressed and
> mouseReleased methods to check for mouse events because I have
> experienced problems with the click handling. Maybe a stateful
> Listener implementation where you have a flag for the handled state
> could help.
It may fix others issues but not the problem we have now. I tested your
suggestion without success:
public void mousePressed(MouseEvent e) {
            handlePopup(e);
        }

        public void mouseReleased(MouseEvent e) {
            System.out.println("released: "+e.isPopupTrigger());
         //returns false systematicaly in catalog view for instance when
using right click... I can"t understand why
            if (e.isPopupTrigger()){
                handlePopup(e);
            }
            else{
                //left click handle
            }
        }

//and no mouseClicked() method

the point is that, for unknown reasons, in some cases, the
isPopupTrigger() method always returns false (reproduced under linux and
it may be your problem cause in the device view, please tell me if my
fix fixed it)

Thanks
>
> BTW the Tutorial
> (http://java.sun.com/docs/books/tutorial/post1.0/ui/mouselistener.html)
> states
>
>  " Release the mouse button.
> You will see a mouse-released event. If you did not move the mouse, a
> mouse-clicked event will follow."
>
>>
>> BTW, if you are interested into testing jarbundler integration, please
>> follow this ticket: https://trac.jajuk.info/ticket/603
>
> Thanks, will check that out later.
>
>>
>> Thanks for your help.
>
> No problem.
>
>>
>> Bertrand
>>>
>>> HTH
>>>
>>> Daniel
>>
>> --Bertrand FLORAT
>> ber...@fl...
>> http://www.florat.net
>> PGP keyserver:  pgp.mit.edu
>>
>>


-- 
Bertrand FLORAT         
ber...@fl...
http://www.florat.net
PGP keyserver:  pgp.mit.edu