tal icon indicating copy to clipboard operation
tal copied to clipboard

Show/hide methods on widget do not have return values

Open bastuijnman opened this issue 8 years ago • 3 comments

Hi guys,

antie/widgets/widget contains both a show and hide method, which according to the JSDoc should return boolean values for whether or not the animation was called. These methods, however, do not return any explicit value (except for the default undefined).

My proposal would be to return the animation reference so developers could cancel show/hide animations when needed, is this something which would be agreeable with you guys? In that case I can make a PR for this.

Kind regards,

Bas Tuijnman 24i

bastuijnman avatar Sep 26 '16 09:09 bastuijnman

Hi Bas,

This is fine from our point of view, so feel free to raise a PR.

Ideally, the animation object returned would be newly created object (following functional principles) but this is not a must.

Tom

tsadler1988 avatar Sep 26 '16 12:09 tsadler1988

Hi Tom,

In this case we can just do return device.hideElement(options) right?

Cheers,

Bas

bastuijnman avatar Sep 26 '16 12:09 bastuijnman

That looks right to me yes.

tsadler1988 avatar Sep 26 '16 13:09 tsadler1988

We have deprecated this project and there are no plans for active development going forward.

Please see the deprecation notice.

kukulaka avatar Jan 04 '23 17:01 kukulaka