Patches item #1176212, was opened at 2005-04-04 04:31
Message generated for change (Comment added) made by thekingant
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176212&group_id=235
Category: None
Group: None
Status: Closed
Resolution: Accepted
Priority: 5
Submitted By: Gary Kramlich (amc_grim)
>Assigned to: Luke Schierer (lschiere)
Summary: Link dnd fixes and then some...
Initial Comment:
Em was in #gaim with a crash on dnd urls, so I walked
'em through avoiding the crash. Then we discovered it
was adding it twice, and realized that the second part
was the data for the A tag. And then I found a memory
leak. So I tweaked the patch to use the label that
we're given.
In other words, crash fix, memory leak fix, use the
label for a url if we're given it.
Things to note, selecting a big block of text with
multiple links in it has issues as does selecting
multiple links. I'm not sure how to go about fixing
that, I leave that to someone who knows what the hell
they're doing with dnd, but i got the same results with
firefox and galeon. Selecting multiple links will use
everything that was selected as the label, and the url
for whatever part you grabbed. When selecing text with
multiple links in it, the output depends on how you
selected it and which part you grabbed to drag.
----------------------------------------------------------------------
>Comment By: Mark Doliner (thekingant)
Date: 2005-04-04 08:26
Message:
Logged In: YES
user_id=20979
Should this be applied to HEAD, also?
----------------------------------------------------------------------
Comment By: Luke Schierer (lschiere)
Date: 2005-04-04 07:20
Message:
Logged In: YES
user_id=28833
you rock.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176212&group_id=235
|
