Opened 9 years ago
Closed 9 years ago
#12333 closed bug (fixed)
Tracker dragged selection is backwards
Reported by: | pulkomandy | Owned by: | waddlesplash |
---|---|---|---|
Priority: | normal | Milestone: | R1 |
Component: | Applications/Tracker | Version: | R1/Development |
Keywords: | Cc: | ||
Blocked By: | Blocking: | ||
Platform: | All |
Description
- Select files from a folder
- Drag them to another app, MediaConvert or StyledEdit for example
- The list is reversed, the last file is listed first in the target app
This is especially annoying when dragging an audio cd content to MediaConvert, as the reversed order leads to useless seeks and my CD drive stays at slow speed because of them (it will go full speed only if reading sequentially for a rather long time).
Attachments (1)
Change History (6)
comment:1 by , 9 years ago
by , 9 years ago
Attachment: | 0001-Tracker-Add-dragged-poses-to-the-drag-message-in-the.patch added |
---|
Patch
comment:2 by , 9 years ago
patch: | 0 → 1 |
---|
comment:3 by , 9 years ago
Owner: | changed from | to
---|---|
Status: | new → in-progress |
@TwoFX: Can you please link to the GCI task for this?
The patch looks good to me. I'll double-check it this afternoon & apply it.
comment:4 by , 9 years ago
@waddlesplash: Thanks! The link is https://codein.withgoogle.com/dashboard/task-instances/6598093361381376/ (I only just claimed it because I was still locked in another task).
comment:5 by , 9 years ago
Resolution: | → fixed |
---|---|
Status: | in-progress → closed |
Applied in hrev49927. Thanks!
I had a look at this for the GCI task to pick any ticket and tackle it. The problem is that the drag message is simly populated in the wrong order. This is due to some helper code that is used to populate the message iterating the selected poses backwards for some reason. I am sure that there is a good reason for this, though according to git blame this has not been changed since pulling OpenTracker into the tree. There is a simple way to fix this problem without breaking anything else by simply doing the iteration yourself. I will attach a patch that does exctly that.