diff options
author | Jon Reid | 2012-05-08 12:04:57 -0700 |
---|---|---|
committer | Jon Reid | 2012-05-08 12:04:57 -0700 |
commit | bafa105c753a139847ef5e0bc8070ce0e8d16f77 (patch) | |
tree | e76ad77474ea9436f23e15de9e7069e888dbe3e3 | |
parent | 307d339e45b209dab80ff88196a9f85f8d58f425 (diff) | |
download | ninja-bafa105c753a139847ef5e0bc8070ce0e8d16f77.tar.gz |
Timeline: Bug fix: Account for horizontal scroll when dragging keyframes.
-rw-r--r-- | js/panels/Timeline/TimelinePanel.reel/TimelinePanel.html | 3 | ||||
-rw-r--r-- | js/panels/Timeline/TimelinePanel.reel/TimelinePanel.js | 5 |
2 files changed, 5 insertions, 3 deletions
diff --git a/js/panels/Timeline/TimelinePanel.reel/TimelinePanel.html b/js/panels/Timeline/TimelinePanel.reel/TimelinePanel.html index 8ddc81cd..5c957619 100644 --- a/js/panels/Timeline/TimelinePanel.reel/TimelinePanel.html +++ b/js/panels/Timeline/TimelinePanel.reel/TimelinePanel.html | |||
@@ -20,6 +20,7 @@ | |||
20 | "track_container": {"#": "right_inside"}, | 20 | "track_container": {"#": "right_inside"}, |
21 | "timeline_leftpane" : {"#" : "leftpane_inside"}, | 21 | "timeline_leftpane" : {"#" : "leftpane_inside"}, |
22 | "layer_tracks": {"#": "layer_tracks"}, | 22 | "layer_tracks": {"#": "layer_tracks"}, |
23 | "layout_tracks": {"#": "layout_tracks"}, | ||
23 | "master_track": {"#": "master_track"}, | 24 | "master_track": {"#": "master_track"}, |
24 | "time_markers" : {"#": "timeline_markers"}, | 25 | "time_markers" : {"#": "timeline_markers"}, |
25 | "layerRepetition" : {"@": "repetition1"}, | 26 | "layerRepetition" : {"@": "repetition1"}, |
@@ -211,7 +212,7 @@ | |||
211 | 212 | ||
212 | 213 | ||
213 | <div data-montage-id="layer_tracks" class="tl_layertracks"> | 214 | <div data-montage-id="layer_tracks" class="tl_layertracks"> |
214 | <div class="layout-tracks"> | 215 | <div class="layout-tracks" data-montage-id="layout_tracks"> |
215 | <div data-montage-id="playhead_marker" class="playheadmarker"></div> | 216 | <div data-montage-id="playhead_marker" class="playheadmarker"></div> |
216 | <div data-montage-id="master_track" class="mastertrack"> | 217 | <div data-montage-id="master_track" class="mastertrack"> |
217 | <div data-montage-id="time_bar" class="timebar"></div> | 218 | <div data-montage-id="time_bar" class="timebar"></div> |
diff --git a/js/panels/Timeline/TimelinePanel.reel/TimelinePanel.js b/js/panels/Timeline/TimelinePanel.reel/TimelinePanel.js index 41472359..933ed9cc 100644 --- a/js/panels/Timeline/TimelinePanel.reel/TimelinePanel.js +++ b/js/panels/Timeline/TimelinePanel.reel/TimelinePanel.js | |||
@@ -1580,7 +1580,8 @@ var TimelinePanel = exports.TimelinePanel = Montage.create(Component, { | |||
1580 | } | 1580 | } |
1581 | */ | 1581 | */ |
1582 | //currPos = event.y - (this._dragAndDropHelperOffset - this.user_layers.scrollTop)- 28; | 1582 | //currPos = event.y - (this._dragAndDropHelperOffset - this.user_layers.scrollTop)- 28; |
1583 | currPos = event.x - 277; | 1583 | |
1584 | currPos = (event.x + this.layout_tracks.scrollLeft) - 277; | ||
1584 | 1585 | ||
1585 | // too much or too little? | 1586 | // too much or too little? |
1586 | if (currPos < this.trackRepetition.childComponents[this.draggingTrackId-1]._keyframeMinPosition) { | 1587 | if (currPos < this.trackRepetition.childComponents[this.draggingTrackId-1]._keyframeMinPosition) { |
@@ -1613,7 +1614,7 @@ var TimelinePanel = exports.TimelinePanel = Montage.create(Component, { | |||
1613 | * | 1614 | * |
1614 | */ | 1615 | */ |
1615 | 1616 | ||
1616 | var currPos = event.x - 274, | 1617 | var currPos = (event.x + this.layout_tracks.scrollLeft) - 277, |
1617 | currentMillisecPerPixel = Math.floor(this.millisecondsOffset / 80), | 1618 | currentMillisecPerPixel = Math.floor(this.millisecondsOffset / 80), |
1618 | currentMillisec = 0, | 1619 | currentMillisec = 0, |
1619 | i = 0, | 1620 | i = 0, |