The PRs link in release page will get network error #13063

Closed
opened 2025-11-02 10:29:03 -06:00 by GiteaMirror · 5 comments
Owner

Originally created by @yp05327 on GitHub (May 29, 2024).

Description

IMG_2701
https://gitea.com/gitea/act_runner/releases

I will check it later. Not sure whether this is a bug now.

Gitea Version

latest

Can you reproduce the bug on the Gitea demo site?

No

Log Gist

No response

Screenshots

No response

Git Version

No response

Operating System

No response

How are you running Gitea?

gitea.com

Database

None

Originally created by @yp05327 on GitHub (May 29, 2024). ### Description ![IMG_2701](https://github.com/go-gitea/gitea/assets/18380374/44990265-cbc8-4b14-b786-01567632357e) https://gitea.com/gitea/act_runner/releases I will check it later. Not sure whether this is a bug now. ### Gitea Version latest ### Can you reproduce the bug on the Gitea demo site? No ### Log Gist _No response_ ### Screenshots _No response_ ### Git Version _No response_ ### Operating System _No response_ ### How are you running Gitea? gitea.com ### Database None
GiteaMirror added the topic/uitype/bug labels 2025-11-02 10:29:03 -06:00
Author
Owner

@silverwind commented on GitHub (May 29, 2024):

Does not reproduce for me:

image
@silverwind commented on GitHub (May 29, 2024): Does not reproduce for me: <img width="578" alt="image" src="https://github.com/go-gitea/gitea/assets/115237/439110be-ab61-499c-926c-51fd0daf282e">
Author
Owner

@yp05327 commented on GitHub (May 29, 2024):

I did this on mobile. On mobile, there's no way to display this tooltips unless you click it.
But if you click it, it will go to the PR's page. And when you back to this page, you will see network error.
So I did two things together: show the tooltips (a bit faster) and go to the PR's page.
So for mobile view, maybe we need to allow user click a URL to show the tooltip at the first time if it has a tooltip, then the second time the click will go to the target link.

@yp05327 commented on GitHub (May 29, 2024): I did this on mobile. On mobile, there's no way to display this tooltips unless you click it. But if you click it, it will go to the PR's page. And when you back to this page, you will see network error. So I did two things together: show the tooltips (a bit faster) and go to the PR's page. So for mobile view, maybe we need to allow user click a URL to show the tooltip at the first time if it has a tooltip, then the second time the click will go to the target link.
Author
Owner

@silverwind commented on GitHub (May 30, 2024):

I can follow. When you click a link, it starts loading, but aborts because the page unloads. When you hit the back button, you get a cached page from bfcache which apparently also caches the error popup.

As for solutions, I think we could try to cancel the request before page unload and not show a error in such a case.

@silverwind commented on GitHub (May 30, 2024): I can follow. When you click a link, it starts loading, but aborts because the page unloads. When you hit the back button, you get a cached page from bfcache which apparently also caches the error popup. As for solutions, I think we could try to cancel the request before page unload and not show a error in such a case.
Author
Owner

@yp05327 commented on GitHub (May 30, 2024):

When you click a link, it starts loading, but aborts because the page unloads.

Yes, that's it. 👍

@yp05327 commented on GitHub (May 30, 2024): > When you click a link, it starts loading, but aborts because the page unloads. Yes, that's it. 👍
Author
Owner

@silverwind commented on GitHub (May 30, 2024):

https://github.com/go-gitea/gitea/pull/31181 will fix this and other problems with that popup.

@silverwind commented on GitHub (May 30, 2024): https://github.com/go-gitea/gitea/pull/31181 will fix this and other problems with that popup.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#13063