Bug 3814 - end of mouse drag or text selection erroneously activates links
Summary: end of mouse drag or text selection erroneously activates links
Status: RESOLVED FIXED
Alias: None
Product: Claws Mail
Classification: Unclassified
Component: Plugins/Fancy (show other bugs)
Version: 3.14.1
Hardware: PC Linux
: P3 normal
Assignee: users
URL:
Depends on:
Blocks:
 
Reported: 2017-04-21 22:53 CEST by Robert
Modified: 2017-04-22 01:24 CEST (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Robert 2017-04-21 22:53:11 CEST
Steps to reproduce:
1. optionally: begin holding down alt or control
2. start dragging a link at least outside of it's own bounds-box
3. press any key (or release modifier)
4. press escape to cancel the drag

Expected:
* the dragged link should pop back in place, without further action

Actual:
* the dragged link pops back in place, but
* claws 'activates' the link in the default web browser
* ...sometimes *TWICE*.

I might guess that claws/fancy is reacting to it's own cancelled drag-and-drop event, but the behavior is a bit weird... especially the double-opening-when-trying-to-cancel effect.
Comment 1 users 2017-04-22 01:19:02 CEST
Changes related to this bug have been committed.
Please check latest Git and update the bug accordingly.
You can also get the patch from:
http://git.claws-mail.org/

++ ChangeLog	2017-04-22 01:19:02.632313343 +0200
http://git.claws-mail.org/?p=claws.git;a=commitdiff;h=12831f0f8f2555230f84893083cb14467eaf21b7
Merge: d9d8a5d ab84747
Author: Colin Leroy <colin@colino.net>
Date:   Sat Apr 22 01:19:02 2017 +0200

    Merge branch 'master' of file:///home/git/claws

http://git.claws-mail.org/?p=claws.git;a=commitdiff;h=ab847472955f979cb63bf0a9b576383e51ef5094
Author: Andrej Kacian <ticho@claws-mail.org>
Date:   Sat Apr 22 01:15:30 2017 +0200

    Fancy: simplify open-or-not decision to x,y matching or not
    
    Fixes bug #3814:  end of mouse drag or text selection erroneously
    activates links