WebAV icon indicating copy to clipboard operation
WebAV copied to clipboard

fix: x,y in rect changed when sprite scale to a minimum size

Open caohongzhi1 opened this issue 1 year ago • 3 comments

修复:针对sprite缩放时,如缩放后的尺寸小于最小限制,会出现x和y坐标点漂移的问题。 bug_view

同时为Rect增加scaleFixedCenter属性控制缩放模式:值为 true 时,将固定中心点不变进行缩放;值为 false 时,将固定对角点不变进行缩放。

最终效果如下: fixed1 fixed2

caohongzhi1 avatar Sep 30 '24 03:09 caohongzhi1

🦋 Changeset detected

Latest commit: aa6f197b00bc996e25e8f2f92fc050a04c415c42

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@webav/av-canvas Patch
@webav/av-cliper Patch
@webav/av-recorder Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

changeset-bot[bot] avatar Sep 30 '24 03:09 changeset-bot[bot]

@caohongz 太棒了 两个目的可以拆开吗,一个是 bugfix,一个是新功能。

最好能补充单测, 还要考虑有旋转角度缩放功能是否正常; 固定中心就当成考虑上下左右四个点的缩放是否正常。

hughfenghen avatar Sep 30 '24 07:09 hughfenghen

@caohongz 太棒了 两个目的可以拆开吗,一个是 bugfix,一个是新功能。

最好能补充单测, 还要考虑有旋转角度缩放功能是否正常; 固定中心就当成考虑上下左右四个点的缩放是否正常。

1、有一些关联,我会尽量拆分成两个PR,如果拆分不开我会新开一个Feature的,附带修复bug; 2、可以补充单测,估计会在假期之后,见谅。

caohongzhi1 avatar Sep 30 '24 07:09 caohongzhi1

@caohongz 太棒了 两个目的可以拆开吗,一个是 bugfix,一个是新功能。

最好能补充单测, 还要考虑有旋转角度缩放功能是否正常; 固定中心就当成考虑上下左右四个点的缩放是否正常。

本次PR仅修复缩放bug,添加了带旋转角度的单测,后续另开PR添加新功能。

caohongzhi1 avatar Oct 08 '24 08:10 caohongzhi1

@caohongz 我稍微优化了代码实现,请检查一下

hughfenghen avatar Oct 09 '24 11:10 hughfenghen

@caohongz 我稍微优化了代码实现,请检查一下

@hughfenghen 确实用switch更简洁易懂了 有一个问题,我希望能在等比例缩放时到达最小尺寸限制后保持比例,非等比例模式时就需要分别设置宽高,所以我又进行了一些改动,并优化了单测代码。

caohongzhi1 avatar Oct 10 '24 01:10 caohongzhi1