Closed Bug 859321 Opened 13 years ago Closed 12 years ago

Track events can get start values < 0

Categories

(Webmaker Graveyard :: Popcorn Maker, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: cade, Assigned: thecount)

Details

Attachments

(1 file)

STR: 1. Add a track event (can even be a media clip) 2. Adjust event to start at 0 and end at the duration of the project 3. Start a drag on the event, try to move it further right - i.e. attempt to make it's end > duration 4. Drop it Expected: the event shouldn't move/change (unless it's placed on a new layer) Actual: the event gets a start time that's less than 0 seconds and an end time that's less than duration. Looks like one of our calculations is over correcting by a few pixels somewhere.
Assignee: nobody → schranz.m
Matt, this is this still valid?
It is.
Status: NEW → ASSIGNED
Seemed better to just deal with it in this spot.
Attachment #769038 - Flags: review?(scott)
Comment on attachment 769038 [details] [review] https://github.com/mozilla/popcorn.webmaker.org/pull/82 No way we can use any form of element.clientWidth here so we don't need to hard code in the border pixels? https://developer.mozilla.org/en-US/docs/Web/API/element.clientWidth?redirectlocale=en-US&redirectslug=DOM%2Felement.clientWidth Also, if you drag and drop to the left, you can get duration of the track event to be greater than the media's duration. Finally, no way to stop the little movement it does when you drag it? It probably shouldn't be doing that either. That might actually be an even deeper cause to this. If the dragging limits are checking css elements with borders, this might happen.
Attachment #769038 - Flags: review?(scott) → review-
Attachment mime type: text/plain → text/x-github-pull-request
Assignee: schranz.m → nobody
Status: ASSIGNED → NEW
Assignee: nobody → scott
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: