Skip to content
This repository has been archived by the owner on Jan 17, 2018. It is now read-only.

canvas added in the body #45

Closed
amirping opened this issue Sep 30, 2017 · 5 comments
Closed

canvas added in the body #45

amirping opened this issue Sep 30, 2017 · 5 comments

Comments

@amirping
Copy link

I'am using vibrant.js with angular 4 , and exprience an issues that the canvas didn't removed from the body
image
and i call it like that
`

let v = new Vibrant(changes.toPlayRadio.currentValue.pic);
v.getPalette((err, palette) => {
  var rgb = palette.Vibrant.getRgb()
  this.radioBackground =
  {
    'background' : 'rgb('+rgb[0]+','+rgb[1]+','+rgb[2]+');'
  }
});

`

@LekoArts
Copy link

@amirping

I also got this issue. Did you resolve it? :/

@LekoArts
Copy link

@jariz Any idea?
When I switch pages the canvas doesn't get removed

@jariz
Copy link
Owner

jariz commented Oct 27, 2017

  1. If the canvas doesn't get cleaned up, it's likely because vibrant crashed.
  2. You're ignoring the error in your code example.

@LekoArts
Copy link

LekoArts commented Oct 27, 2017

Actually I found the solution I think:
Vibrant-Colors/node-vibrant#46

Sadly he/she isn't maintaining the package anymore and therefore this PR won't be merged anytime soon.

@akfish
Copy link
Collaborator

akfish commented Nov 28, 2017

Hi, developer of node-vibrant here.
I'm back actively maintaining node-vibrant again. @jariz and I decided to deprecate vibrant.js.
This issue is fixed in node-vibrant. Please try it out. :)

@akfish akfish closed this as completed Nov 28, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants