Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Implement fabric.Object#backgroundColor for all classes #883

Closed
kangax opened this issue Sep 30, 2013 · 1 comment · Fixed by #3248
Closed

Implement fabric.Object#backgroundColor for all classes #883

kangax opened this issue Sep 30, 2013 · 1 comment · Fixed by #3248
Assignees
@asturur
Labels
feature

Comments

@kangax
Copy link
Member

kangax commented Sep 30, 2013

Currently only works with text objects

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
@asturur asturur self-assigned this Apr 13, 2016
@asturur
Copy link
Member

asturur commented May 2, 2016

Why backgroundColor and not backgroundFill and being generic about the fill?

color-gradient-pattern.
Ok for image, paths, polygon, polyline, groups

But that would be strange for
circle, line, group with bacgrounded-objects

useless for
rects

@asturur asturur added this to the 1.6.5 milestone Sep 11, 2016
@asturur asturur mentioned this issue Sep 11, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@asturur asturur

Labels
feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

move backgroundColor to all classes fabricjs/fabric.js
2 participants
@kangax @asturur