aboutsummaryrefslogtreecommitdiff
path: root/js/tools/PenTool.js
Commit message (Collapse)AuthorAge
* More consistent local and global mouse coords which solves a bug with the ↵Pushkar Joshi2012-05-29
| | | | Pen minus subtool not removing 1 anchor paths
* allow changes in the pen subtool in options to be seen by the pen tool code ↵Pushkar Joshi2012-05-22
| | | | | | (can now select the pen plus, pen minus subtools) AND add keyboard shortcut for brush tool
* Merge branch 'master' into pentoolPushkar Joshi2012-05-21
|\
| * put color chips in the inkbottle and fill toolJohn Mayhew2012-05-10
| |
| * Merge branch 'master' of github.com:Motorola-Mobility/ninja-internal into ↵John Mayhew2012-05-07
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | WorkingBranch Conflicts: js/components/layout/tools-list.reel/tools-list.html js/components/layout/tools-properties.reel/tools-properties.html js/components/tools-properties/brush-properties.reel/brush-properties.html js/components/tools-properties/fill-properties.reel/fill-properties.html js/components/tools-properties/pen-properties.reel/pen-properties.html js/components/tools-properties/pencil-properties.reel/pencil-properties.html js/components/tools-properties/selection-properties.reel/selection-properties.html js/components/tools-properties/shape-properties.reel/shape-properties.html js/components/tools-properties/tag-properties.reel/tag-properties.html js/components/tools-properties/text-properties.reel/text-properties.html
| * | First round of moving color chips into the sub tools. Shape and Pen tool now ↵John Mayhew2012-05-03
| | | | | | | | | | | | have chips in the sub tool bar. Still need to complete adding chips to the Brush tool and finalizing the subtool bar layout to our spec for all of the subtools.
* | | handle anchor point deletion correctly (does not yet fully work for ↵Pushkar Joshi2012-05-11
| | | | | | | | | | | | PEN-MINUS subtool)
* | | code cleanup and starting the subtool functionalityPushkar Joshi2012-05-11
| | |
* | | Merge branch 'master' into pentoolPushkar Joshi2012-05-07
|\ \ \ | | |/ | |/|
| * | Refactor creating elements. Removed makeNJElement and separated the model ↵Valerio Virgillito2012-05-03
| |/ | | | | | | | | | | creation Signed-off-by: Valerio Virgillito <valerio@motorola.com>
* | handle delete more correctly than before (selected subpaths are ↵Pushkar Joshi2012-05-04
| | | | | | | | deleted)....this is buggy when the second anchor of a two-anchor subpath is deleted (since the first anchor needs to be converted back into stage world coords)
* | flip the action of the alt key when modifying anchor handles (this ↵Pushkar Joshi2012-05-02
| | | | | | | | essentially forces the need for a keyboard...must be addressed by a fix later on)
* | Fix for 1524 Pen: Unable to add anchor to a closed pathPushkar Joshi2012-04-27
| | | | | | | | ---behavior is unchanged, but the realtime feedback for the tool (i.e. mouse cursor) will correctly reflect that a click after closing a path will start a new path, not add to that path
* | add event handlers for open, close, and switch document to pen tool, fixes:Pushkar Joshi2012-04-27
| | | | | | | | 1532 Pen: Anchor points get carried over to the new document
* | Fix for 1525 Pen: "Uncaught RangeError: Maximum call stack size exceeded".Pushkar Joshi2012-04-27
|/ | | | (check if the second mouse click actually selects the existing first anchor point)
* re-compute and store the center of the subpath in local coordinates upon ↵Pushkar Joshi2012-04-24
| | | | file open
* allow the addition of the pen tool to the timelinePushkar Joshi2012-04-23
|
* get the canvas plane matrix before multiplying it with the translation due ↵Pushkar Joshi2012-04-19
| | | | to shift in center of rotation
* correctly maintain canvas position when drawing subpaths off the XY plane ↵Pushkar Joshi2012-04-19
| | | | (was a bug with starting drawing in those planes)
* for subpaths that have not been initialized with a canvas yet (i.e. no local ↵Pushkar Joshi2012-04-19
| | | | coordinates), use stage world mouse positions to translate the anchor points while dragging
* store the canvas center (in local space) per each subpath, instead of ↵Pushkar Joshi2012-04-19
| | | | tracking it by the pen tool. This fixes the bug where the canvas transform was incorrectly applied when the stroke width was changed
* turn off snapping for the pen tool (with an option to turn it back on if we ↵Pushkar Joshi2012-04-19
| | | | decide to, later)
* fix a bug with closing the subpathPushkar Joshi2012-04-18
|
* re-enable dragging and editing of anchor point handles for bezier path ↵Pushkar Joshi2012-04-18
| | | | editing in 3D
* compute the translation to new center of bounding of the subpath --- this ↵Pushkar Joshi2012-04-18
| | | | preserves the correct transformation applied to the subpath. This is a good snapshot of the polyline drawing in 3D (with some minor bugs).
* allow drawing polylines in YZ and XZ planes as well as XY. In those planes, ↵Pushkar Joshi2012-04-18
| | | | still have bug with shift when the bbox of the path grows (due to missing translation)
* Fairly stable version of polyline drawing code that works for paths on XY, ↵Pushkar Joshi2012-04-17
| | | | | | YZ, XZ planes. Current bugs: if the new point increases the bbox of the canvas, the entire canvas is shifted (due to inconsistency with center of rotation) AND the first two points added to planes other than XY have incorrect local coordinates (due to missing local coordinates)
* took out bad merge line and use different function for converting points ↵Pushkar Joshi2012-04-17
| | | | from local to global coords (the latter should be a temporary change as I will significantly change this)
* Merge branch 'master' into pentoolPushkar Joshi2012-04-17
|\ | | | | | | | | Conflicts: js/tools/PenTool.js
| * -webkit-transform-style needs to be set to preserve-3d for shapes regardless ↵Nivesh Rajbhandari2012-04-05
| | | | | | | | | | | | of whether they are 2d shapes or WebGL shapes. Signed-off-by: Nivesh Rajbhandari <mqg734@motorola.com>
| * Removing fill, stroke, fillMaterial and strokeMaterial from the shapeModel ↵Nivesh Rajbhandari2012-04-05
| | | | | | | | | | | | cache because they can easily get out of sync with the actual colors and materials. Instead, we will always query the object for these values. Signed-off-by: Nivesh Rajbhandari <mqg734@motorola.com>
| * Merge branch 'refs/heads/ToolFixes' into WebGLMaterialsNivesh Rajbhandari2012-04-04
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: js/document/templates/montage-html/default_html.css js/mediators/element-mediator.js js/panels/properties.reel/properties.js js/tools/BrushTool.js js/tools/LineTool.js js/tools/PenTool.js js/tools/SelectionTool.js js/tools/ShapeTool.js js/tools/TranslateObject3DTool.js Signed-off-by: Nivesh Rajbhandari <mqg734@motorola.com>
| * | WebGL needs to have 3d styles to render.Nivesh Rajbhandari2012-03-22
| | | | | | | | | | | | Signed-off-by: Nivesh Rajbhandari <mqg734@motorola.com>
* | | Snapshot for pen tool before merging with master....can draw polylines (no ↵Pushkar Joshi2012-04-17
| | | | | | | | | | | | prev, next) handles and hit testing in 3D. Update of the canvas with a transformation on it will modify the canvas incorrectly, and will be completed after merging with master
* | | re-enable some hit testing and change the code for obtaining 3D (stage ↵Pushkar Joshi2012-04-16
| | | | | | | | | | | | world) point when there is no canvas yet
* | | limit the size of the subpath canvas by ignoring the last point added that ↵Pushkar Joshi2012-04-16
| | | | | | | | | | | | tool the canvas size over the specified limit
* | | Allow the path stroke width to be changed without causing a drift in the ↵Pushkar Joshi2012-04-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | canvas position AND some code cleanup (remove canvas left and top tracking for each subpath) AND add flags for the specifying in what coordinate space we're sampling the subpath
* | | Still buggy version of pen tool that can create polylines in 3D space ↵Pushkar Joshi2012-04-13
| | | | | | | | | | | | correctly (except for incorrect canvas translation). Using cleaned up and re-factored version of the code. Not suitable for testing.
* | | Use local coordinates for rendering and hit testing only when the path ↵Pushkar Joshi2012-04-12
| | | | | | | | | | | | canvas exists....AND...good state for code that works in 3D as long as the path was originally created in XY plane of the stage
* | | create paths in 3D...still buggy if starting from scratch in planes other ↵Pushkar Joshi2012-04-11
| | | | | | | | | | | | than XY
* | | correctly add points to open paths on transformed canvasesPushkar Joshi2012-04-11
| | |
* | | dragging anchor points in 3D (seems to work ok for existing paths --- the ↵Pushkar Joshi2012-04-11
| | | | | | | | | | | | left and top change causes the rest of the points to move, which *may* be unavoidable), still has bugs with NEW points being placed at wrong position
* | | Use local coordinates to pick a point within the path (works correctly even ↵Pushkar Joshi2012-04-11
| | | | | | | | | | | | with canvas and/or stage transformation). Dragging does not yet work in case of canvas transformation
* | | compute and store local coordinates for all anchor points and their control ↵Pushkar Joshi2012-04-10
| | | | | | | | | | | | handles separately, AND more hit testing with local coordinates
* | | snapping feedback for pen tool, correct for subpaths that may lie on rotated ↵Pushkar Joshi2012-04-09
| | | | | | | | | | | | canvas, and with a rotated view
* | | improved comment about plane matrix (no real change)Pushkar Joshi2012-04-06
| | |
* | | Don't update the plane matrix of subpaths that already have a plane matrixPushkar Joshi2012-04-06
| | |
* | | code cleanup and working version of selected path anchor point and overlay ↵Pushkar Joshi2012-04-06
| | | | | | | | | | | | display for new paths
* | | complete version of the code for rendering selected subpath anchor points ↵Pushkar Joshi2012-04-06
| | | | | | | | | | | | and path overlay (code needs cleanup)...still buggy if creating new paths
* | | Merge branch 'master' into pentoolPushkar Joshi2012-04-04
|\ \ \ | | |/ | |/| | | | | | | Conflicts: js/tools/PenTool.js