[FEATURE]: Add a Configuration Table

[ghost]

Contact Details

https://devforum.roblox.com/u/vscplays/summary

Is your feature request related to a problem?

I have an problem when I was adding some values, It makes the code harder for me to read

these are the properties

time: number?,
	style: Enum.EasingStyle?,
	direction: Enum.EasingDirection?,
	repeatCount: number?,
	reverses: boolean?,
	delayTime: number?

Describe the solution you'd like

I would like to see the configuration table of these properties

time: number?,
	style: Enum.EasingStyle?,
	direction: Enum.EasingDirection?,
	repeatCount: number?,
	reverses: boolean?,
	delayTime: number?

Describe alternatives you've considered

No response

Additional context

No response

Code of Conduct

• I agree to follow this project's Code of Conduct

[RyanLua]

From what your stating you would prefer to have the constructor be changed from

new(parent: GuiObject, time: number?, style: EasingStyle?, direction: EasingDirection?, repeatCount: number?, reverses: boolean?, delayTime: number?)

to

new(parent: GuiObject, tweenInfo: TweenInfo?)

or something similar. The TweenInfo data type already can be used instead of a table.

Shime aims to be as familiar and easy to use as possible so it would make sense to borrow TweenInfo table, which is very much in scope of Shime.

Currently as it stands if I do change the constructor, it would break backwards compatibility with the previous, including LTS releases of Shime.

[ghost]

Yeah that is better ;)

For now I gotta close this

I found how to fix this

function Shime.new(parent: GuiObject, tweenInfo: TweenInfo?)
	tweenInfo = tweenInfo or TweenInfo.new(1, Enum.EasingStyle.Linear, Enum.EasingDirection.InOut, -1, false, 0)

	local self = setmetatable({}, Shime)

	-- Create the shimmer frame and animation
	local shimmer = createShimmer(parent)
	self._frame = shimmer
	self._gradient = shimmer:FindFirstChildOfClass("UIGradient")
	self._corner = shimmer:FindFirstChildOfClass("UICorner")

	-- Create the tween
	self._tween = TweenService:Create(
		self._gradient,
		tweenInfo,
		{ Offset = Vector2.new(1, 0) }
	)

	-- Setup tween completion callback. Under default constants this will never be called as the tween will repeat infinitely
	self._tween.Completed:Connect(function()
		self:_TweenCompleted()
	end)

	return self
end

[RyanLua]

I didn't push any commits? I'm not sure how I solved this issue but alright.

[ghost]

You mentioned this:

From what your stating you would prefer to have the constructor be changed from

new(parent: GuiObject, time: number?, style: EasingStyle?, direction: EasingDirection?, repeatCount: number?, reverses: boolean?, delayTime: number?)

to

new(parent: GuiObject, tweenInfo: TweenInfo?)

or something similar. The TweenInfo data type already can be used instead of a table.

[RyanLua]

You mentioned this:

From what your stating you would prefer to have the constructor be changed from

new(parent: GuiObject, time: number?, style: EasingStyle?, direction: EasingDirection?, repeatCount: number?, reverses: boolean?, delayTime: number?)

to

new(parent: GuiObject, tweenInfo: TweenInfo?)

or something similar. The TweenInfo data type already can be used instead of a table.

I'm not planning to introduce a new constructor yet as it will break all pre-existing ones and require me to update all documentation for a small change which gives little benefits.

So I don't get why this is marked as completed.