Fix circular spin animation direction

[lutinglt]

Wait for the status icon to rotate clockwise instead of counterclockwise

before:

after:

This can be merged to 1.25

[wxiaoguang]

Please read and fix the comment

[lutinglt]

Please read and fix the comment

Now, we have our own icon that can rotate clockwise (

[wxiaoguang]

@silverwind what do you think about the new change? It looks good to me.

[lutinglt]

@silverwind what do you think about the new change? It looks good to me.

I have a better idea. 'octicon sync' is a counterclockwise icon used to represent the state of rollback or rerun. If it is a rollback or rerun, we should use a counterclockwise icon and a counterclockwise rotation animation. If it is ongoing, first-time, or one-time execution, we should use a clockwise icon and a clockwise rotation animation. This can be a bit troublesome because I am not very clear about the specific purpose of some pages.

[wxiaoguang]

Maybe no need to make it that complex. When it is a static icon, I guess few people would really look carefully at it to see whether it is clockwise or counterclockwise

[lutinglt]

Maybe no need to make it that complex. When it is a static icon, I guess few people would really look carefully at it to see whether it is clockwise or counterclockwise

Emmm, Maybe it's still quite obvious?

[silverwind]

1. Maybe we should just use the dot-in-circle icon in all cases where we had animate the other one, which would make it never spin, and therefor we would not need to have the flipped icon being animated.
2. Can we rename the animation to rotate-clockwise?

[lutinglt]

1. Maybe we should just use the dot-in-circle icon in all cases where we had animate the other one, which would make it never spin, and therefor we would not need to have the flipped icon being animated.
2. Can we rename the animation to rotate-clockwise?

I agree, but I can't see the specific page. It seems like there's no problem looking at the code? If possible, I will modify and submit it like this.

[silverwind]

Should be safe to replace "blindly" if the icon size is the same.

[lutinglt]

The viewBox for octicon-sync and gitea-running is the same. I will replace the octicon-sync icon with gitea-running for the circular-pine animation and rename the circular-pine animation class name to rotate-clockwise