The status icon of the Action step is consistent with GitHub

[lutinglt]

#35616
Before:
running:

waiting:

---

After:
running:

waiting:

gitea-running.svg is not an icon from the @ primer/octicon library, extracted from the Github page. Github did not assign a clear class name to this icon

[wxiaoguang]

The SVG source should be in web_src/svg and run make svg

[lutinglt]

@wxiaoguang Sorry, I'm not very familiar with Git and don't know how to backport it to 1.25 😖

[silverwind]

@wxiaoguang Sorry, I'm not very familiar with Git and don't know how to backport it to 1.25 😖

Assuming no merge conflicts, backport is performed automatically after merge and you don't need to do anything.

[silverwind]

As a future improvement, we could animate gitea-running.svg using SVG SMIL.

[lutinglt]

As a future improvement, we could animate gitea-running.svg using SVG SMIL.

I tried to implement it with AI and it looks good, but AI seems to indicate that this technology has been abandoned? CSS animations should be used for implementation, and Github also uses CSS for implementation.

<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 16 16" width="16" height="16">
  <path fill="none" stroke="currentColor" stroke-width="2" d="M3.05 3.05a7 7 0 1 1 9.9 9.9 7 7 0 0 1-9.9-9.9Z" opacity=".5"></path>
  <path fill="currentColor" fill-rule="evenodd" d="M8 4a4 4 0 1 0 0 8 4 4 0 0 0 0-8Z" clip-rule="evenodd"></path>
  <path fill="currentColor" d="M14 8a6 6 0 0 0-6-6V0a8 8 0 0 1 8 8h-2Z">
    <animateTransform attributeName="transform" type="rotate" from="0 8 8" to="360 8 8" dur="1s" repeatCount="indefinite" />
  </path>
</svg>

[silverwind]

No, SMIL is not abandoned. A few years ago Google wanted to kill it but they have since reverted their stance and it'll stay as part of the web platform. I like it because it enables self-contained SVG animations.

[lutinglt]

No, SMIL is not abandoned. A few years ago Google wanted to kill it but they have since reverted their stance and it'll stay as part of the web platform. I like it because it enables self-contained SVG animations.

Do we need to add this part to this merger? I saw on GitHub that the merge has been locked.

If using SVG SMIL, the class name of the animation should also be removed, and overlaying CSS animation will accelerate rotation.

[wxiaoguang]

No need to make it that complex.

[silverwind]

Let's keep it to CSS. It at least is more broadly supported than SMIL (I'm thinking of browsers like Palemoon or Ladybird).

[lutinglt]

Got it, Gitea's level of similarity to GitHub has increased by another 1% 😄