[kde-doc-english] [kscreengenie] /: Abort On-Click on non-left clicks

[Boudhayan Gupta]

On 7 July 2015 at 22:24, Alexander Potashev <aspotashev at gmail.com> wrote:
> 2015-07-07 19:28 GMT+03:00 Boudhayan Gupta <me at baloneygeek.com>:
>> On 7 July 2015 at 21:52, Alexander Potashev <aspotashev at gmail.com> wrote:
>>> 2015-07-07 19:04 GMT+03:00 Boudhayan Gupta <me at baloneygeek.com>:
>>>> +                               <para>Enabling the <guilabel>On Click</guilabel> checkbox overrides the delay. When this checkbox is enabled, pressing the <guibutton>Take New Screenshot</guibutton> (<keycombo action="simul">&Ctrl;<keycap>N</keycap></keycombo>) button hides the &kscreengenie; window and changes the mouse cursor to a crosshair. The screenshot is captured when the mouse is left-clicked, or aborted if the mouse is right-clicked. Note that you cannot interact with the desktop using the mouse while the cursor is a crosshair, but you can use the keyboard.</para>
>>>
>>> Hi Boudhayan,
>>>
>>> You are not stating the behaviour when middle or other mouse button is
>>> clicked. And what happens if I roll the mouse wheel?
>>>
>>> --
>>> Alexander Potashev
>>
>> Hi Alexander,
>>
>> The technical detail of the implementation is like this: if it's
>> XCB_BUTTON_MASK_1 (which is left-click), take the shot, for any other
>> button abort. It's like this because XCB_BUTTON_MASKs differ from
>> mouse to mouse - i.e., on a two button mouse the right click is
>> XCB_BUTTON_MASK_2, while on a three-button mouse it is
>> XCB_BUTTON_MASK_3 with MASK_2 being the middle button.
>>
>> Scrolling the mouse wheel also generates button press events, so that
>> also aborts the shot.
>>
>> I'm not sure how to put so much information into a document meant for
>> users, so if you want to rephrase it, go ahead.
>
> Boudhayan,
>
> I don't think it's reasonable to cancel the shot by mouse wheel. If we
> somehow ignore the scrolling event, then the doc could say "left click
> take a screenshot, clicking any other mouse button cancels".
>

I've pushed out code to filter mouse scroll events. You can edit the
doc as you see fit.

-- Boudhayan