fix enabled

[The-Redhat]

Hey.. and another pull request.

I tested the new version v1.0.1 and the enabled property didn't worked for me.
For example:

In my example the paint method is called. I fixed it. Moreover paint now only calls saveLayer (heavy method), when it's necessary.

[hnvn]

I think we are misunderstanding about enabled feature. In my view, I am implementing it as a way to stop translating animation, I don't remove all gradient blending, so the animation can be resumed from the state it is stopped. As I read from your codes, you want to remove all gradient blending and just draw the child, do you?

[The-Redhat]

Yes. I try to follow the 5th point of the performance considerations.

[The-Redhat]

Maybe we could use enabled the way you suggest it and when the gradient colors are null only paint the child.

[Zazo032]

@hnvn could this be merged and published? This is the desired behaviour we're looking for in our app

[gmaggio]

@hnvn: I really need this ability to remove all gradient blending and just draw the child. I think it will come in handy for a lot of developers.

Anyway, wouldn't it be easier to just do this, @The-Redhat @hnvn:

  @override
  Widget build(BuildContext context) {
    return AnimatedBuilder(
      animation: _controller,
      child: widget.child,
      builder: (BuildContext context, Widget child) {
        return !widget.enabled
            ? child
            : _Shimmer(
                child: child,
                direction: widget.direction,
                gradient: widget.gradient,
                percent: _controller.value,
              );
      },
    );
  }

Or maybe, just in case, have another property for this feature, like hideOnDisabled maybe, so we can have a choice as to how we want it to behave.

[The-Redhat]

Yes that could be a simple solution. I would suggest that you open a new pr, because I'm currently on vacation.

[gmaggio]

@The-Redhat Okay... I added the PR here: #32