Incorrect documentation for `setPosition` method in `AnimationPlayer` interface

[olfek]

Which @angular/* package(s) are the source of the bug?

animations

Is this a regression?

No

Description

angular/packages/animations/src/players/animation_player.ts

Lines 76 to 80 in 950a554

	/**
	* Sets the position of the animation.
	* @param position A 0-based offset into the duration, in milliseconds.
	*/
	setPosition(position: number): void;

angular/packages/animations/src/players/animation_player.ts

Lines 197 to 199 in 950a554

	setPosition(position: number): void {
	this._position = this.totalTime ? position * this.totalTime : 1;
	}

As you can see, position: number is used as a 0-1 fractional value.

But the comment says:

A 0-based offset into the duration, in milliseconds.

Please provide a link to a minimal reproduction of the bug

No response

Please provide the exception or error you saw

No response

Please provide the environment you discovered this bug in (run ng version)

No response

Anything else?

No response

[olfek]

Being able to set the this._position unit (millisecond position) would be better, so maybe the documentation is correct, and it is actually the code that is incorrect?

[muhammadawaisshaikh]

I need to work on it. thanks

[muhammadawaisshaikh]

this is done already, please close this @olfek

PR Closed: #57927

[olfek]

closing. fixed by - 2f5f898

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}