Opened 11 years ago

Last modified 20 months ago

#1451 assigned bug

Right mouse click over Nick lister in Bowser: Setup doesn't work in Bowser 1.1.1

Reported by: ackcontrols Owned by: nobody
Priority: normal Milestone: R1
Component: Applications Version: R1/pre-alpha1
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

When running Bowser 1.1.1 (downloaded from BeBits), I cannot add a Nick by pressing the right mouse key. Works under BeOS R5, but not under Haiku. Hope this helps someone. When right mouse clicking, a menu should appear with the options Add, Remove, Load.

Change History (4)

comment:1 Changed 8 years ago by mmadia

Can you retest this on R1/Alpha 2 or even a nightly image?

comment:2 Changed 6 years ago by siarzhuk

The problem is that Bowser uses MessageFilter to handle menu and keyboard events in the list of nicks. That filter is created with following flags:

http://bowser.cvs.sourceforge.net/viewvc/bowser/Bowser/src/SetupWindow.cpp?revision=1.13&view=markup

1217	NickFilter::NickFilter (SetupView *sv)
1218	        : BMessageFilter (B_PROGRAMMED_DELIVERY, B_LOCAL_SOURCE),
1219	          view (sv)
1220	{
1221	}

And changing B_LOCAL_SOURCE to B_ANY_SOURCE restores function of context menu. Looks like Haiku has another meaning about B_LOCAL_SOURCE than BeOS had.

BeBook say about the 'source' parameter:

source Specifes whether the sender of the message must be local vis-a-vis this app (B_LOCAL_SOURCE), remote (B_REMOTE_SOURCE), or either (B_ANY_SOURCE). The default is B_ANY_SOURCE.

In my opinion messages issued by user's mouse click should be counted as remote ones, so this issue looks like BeOS error.

Do you have any opinions about this, gentlemen?

comment:3 Changed 6 years ago by axeld

I would agree with that - the only remaining question is if we want to be compatible or not (for example, we could force B_ANY_SOURCE for all applications linked for BeOS).

comment:4 Changed 20 months ago by axeld

Owner: changed from axeld to nobody
Status: newassigned
Note: See TracTickets for help on using tickets.