Commit graph

292 commits

Author SHA1 Message Date
asturur
20df73c362 Update polyline.class.js
never touch too much.
2014-07-18 17:49:29 +02:00
asturur
c58abe2698 Update polygon.class.js
Never touch too much.
2014-07-18 17:48:28 +02:00
asturur
2495d58523 Update line.class.js 2014-07-18 16:28:29 +02:00
asturur
f610ffe9de Update path.class.js 2014-07-18 16:27:29 +02:00
asturur
4db2d6c5ed Update text.class.js 2014-07-18 16:24:34 +02:00
asturur
3bf7fbc619 Update text.class.js 2014-07-18 16:24:15 +02:00
asturur
0c36a1a7cc Update polygon.class.js 2014-07-18 16:13:46 +02:00
asturur
58e318f214 Update polyline.class.js 2014-07-18 16:12:39 +02:00
kangax
3df3c396a2 Fix unit tests 2014-07-18 13:11:18 +02:00
Stefan Kienzle
6b5f049bb0 JSDoc + JSCS tweaks - Part 2 2014-07-18 11:16:23 +02:00
Stefan Kienzle
b34387d10e JSDoc + JSCS tweaks - Part 1 2014-07-17 16:18:57 +02:00
Mike Shaver
825bec65df More doc-comment fixes. 2014-07-15 15:59:53 -07:00
Mike Shaver
095ee86e37 Fix some doc comment errors that were preventing build. 2014-07-15 15:34:06 -07:00
asturur
a453ba83d7 Update ellipse.class.js
Fixes border without killing opacity. Opacity need to be moved in render function maybe, not in _render.
2014-07-10 12:09:47 +02:00
Tom French
6b8ce5c1ce JSHint 2014-07-05 18:06:21 +01:00
Tom French
300de3be0d Correct rendering of active group 2014-07-04 20:07:04 +01:00
Tom French
6fbd5df49c Fix FromObject 2014-07-04 11:29:48 +01:00
Tom French
a4129c31e8 Merge remote-tracking branch 'upstream/master'
Conflicts:
	dist/fabric.min.js
	dist/fabric.min.js.gz
	dist/fabric.require.js
	src/canvas.class.js
2014-07-03 16:02:43 +01:00
Chris Buergi
9a867f893f Fire 'selection:changed' on IText object.
Previously the 'text:selection:changed' was only fired on the canvas, but not on the object.
2014-07-03 10:02:47 +02:00
Chris Buergi
dda68b44dd Render IText correctly when only fontWeight or fontStyle changes
Fixes rendering of characters that only have the styles "fontWeight" and/or "fontStyle" set. Previously _hasStyleChanged() did not detect a change if only one of those two styles have been set.
2014-06-30 16:19:35 +02:00
Stefan Kienzle
50e3fd717c Merge pull request #1427 from asturur/shadow-return-null
Update shadow.class.js - `objects.setShadow(null)` removes the sahdow
Closes #1422
2014-06-24 15:48:16 +02:00
asturur
22b0149e49 Update object.class.js
as suggested by Kienz.
2014-06-24 15:38:14 +02:00
kangax
492253e5ca JSCS tweaks. Down to 114 failures. 2014-06-24 14:12:17 +02:00
kangax
715fda8e94 Revert "Update path.class.js"
This reverts commit 77dd88569d.
2014-06-21 19:23:44 +02:00
kangax
62eb4e39f5 Revert "Build dist"
This reverts commit ac006b837f.
2014-06-21 19:15:10 +02:00
kangax
ac006b837f Build dist 2014-06-21 19:10:20 +02:00
asturur
77dd88569d Update path.class.js
Modified path positioning, fixes svg 170, debian logo, change some errors in different errors.
2014-06-21 17:03:54 +02:00
asturur
8294fd42ab Update circle.class.js
Same kind of modification from ellipse.
2014-06-21 10:37:31 +02:00
asturur
beb3fc205f Update ellipse.class.js
just extra space.
2014-06-21 10:31:04 +02:00
asturur
4729d104c8 Update ellipse.class.js
As crazy as it looks like. Translate the ellipse just if it doesn't have a transformMatrix.
I have the feeling that the problem is somewhere else, but i cannot fix it different way.
2014-06-21 10:29:22 +02:00
asturur
1257754d5b Update circle.class.js
Forgot those changes.
2014-06-20 09:17:34 +02:00
asturur
402c2a6f3a Update ellipse.class.js
Fixed some error i could not find but stopped the js to work.
2014-06-20 09:07:35 +02:00
asturur
efa6579450 Update path.class.js 2014-06-19 23:47:21 +02:00
asturur
fd658ce29b remove path positioning fixes
in different branch.
2014-06-19 23:46:06 +02:00
asturur
9c444b4ecb Path positioning
This path positioning change renders correctly the svg 170 ( the many red triangles ) and change the visualization of issue #1363. Need extensive testing. Consider that viewbox has to be implemented better, so don't take too much negatively the errors on svgs that have viewbox with negative numbers.

if this is not a solution i hope at least it can inspire someone
2014-06-19 23:33:12 +02:00
asturur
2dc5e298a3 Update ellipse.class.js 2014-06-19 21:38:26 +02:00
asturur
66d6b633ba Update ellipse.class.js 2014-06-19 21:37:33 +02:00
asturur
3f47559589 Start to fix the position of ellipses
Till this library won't use ctx.ellipse() this weird transformation will be a huge mess.
This little fix imroves position of not transformed ellipses.
You can see the fixes on the example i will post below.
It needs improvement to work with other transformMatrix.
2014-06-19 21:05:39 +02:00
asturur
190973f6c9 Update ellipse.class.js
Always transparency rendering related.
2014-06-19 17:14:36 +02:00
asturur
8482c1c29b Update polygon.class.js
Apply opacity property to poligon class.
2014-06-19 10:42:55 +02:00
asturur
572038b0e5 Update path.class.js
apply opacity property on paths.
2014-06-19 10:42:05 +02:00
Tom French
5f8e0ee6ee Fixes for group coordinates (coordinates now also generated before group is added to canvas) 2014-06-13 11:23:13 +01:00
Tom French
b13e5763b6 Calculate group coordinates when initializing 2014-06-12 12:37:20 +01:00
Tom French
cd4d8b2c17 Fix group.hasMoved 2014-06-12 10:13:08 +01:00
Jim Rodovich
1201cfb1e1 Make closepath commands update current x/y coordinates.
Per the SVG spec,
> If a "closepath" is followed immediately by any other command, then
> the next subpath starts at the same initial point as the current
> subpath.
2014-06-06 13:50:54 -05:00
Tom French
8c826a3204 Fixes for controlsAboveOverlay 2014-06-06 17:36:17 +01:00
Jim Rodovich
ff04efdc76 Don't ever draw lines to explicit M/m commands.
#1365 converted multiple M/m coordinates to L/l commands when importing paths. The `_render` function was already attempting to connect those coordinates, but that's no longer necessary as the only consecutive M/m commands in `_render` were explicitly defined as M/m commands.
2014-06-06 10:31:54 -05:00
Tom French
b4746e6bf2 Merge remote-tracking branch 'upstream/master' 2014-05-31 17:33:28 +01:00
Tom French
0d8db88e71 Merge remote-tracking branch 'upstream/master'
Conflicts:
	.gitignore
	dist/fabric.js
	dist/fabric.require.js
	src/brushes/circle_brush.class.js
	src/canvas.class.js
	src/mixins/canvas_events.mixin.js
	src/mixins/object_geometry.mixin.js
	src/mixins/object_interactivity.mixin.js
	src/shapes/group.class.js
	src/shapes/image.class.js
	src/shapes/object.class.js
	src/shapes/path_group.class.js
	src/shapes/text.class.js
2014-05-31 17:29:55 +01:00
Jim Rodovich
6f8444c86a Convert extra "moveto" coordinates to "lineto" commands.
Per the [SVG spec](http://www.w3.org/TR/SVG11/paths.html#PathDataMovetoCommands):

> If a moveto is followed by multiple pairs of coordinates, the
> subsequent pairs are treated as implicit lineto commands. Hence,
> implicit lineto commands will be relative if the moveto is relative,
> and absolute if the moveto is absolute."
2014-05-30 10:08:03 -05:00