Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The code comment dispeared after resolving/unresolving a comment #28949

Closed
lunny opened this issue Jan 27, 2024 · 8 comments · Fixed by #29050
Closed

The code comment dispeared after resolving/unresolving a comment #28949

lunny opened this issue Jan 27, 2024 · 8 comments · Fixed by #29050
Labels
Milestone

Comments

@lunny
Copy link
Member

lunny commented Jan 27, 2024

Description

The code comment dispeared after resolving/unresolving a comment

Gitea Version

main

Can you reproduce the bug on the Gitea demo site?

Yes

Log Gist

No response

Screenshots

图片

Git Version

No response

Operating System

No response

How are you running Gitea?

gitea.com

Database

None

@lunny lunny added the type/bug label Jan 27, 2024
@yp05327
Copy link
Contributor

yp05327 commented Jan 29, 2024

Can you provide the URL?

@lunny
Copy link
Member Author

lunny commented Jan 29, 2024

@yp05327
Copy link
Contributor

yp05327 commented Jan 29, 2024

I can see it on my phone.
IMG_3301
and on pc
image

@wxiaoguang
Copy link
Contributor

Maybe the reason is: "ShowOutdatedComments" defaults to "false", so you won't see the "outdated" comments.

Maybe it actually should default to true.

image

@lunny
Copy link
Member Author

lunny commented Feb 4, 2024

But the experience is very odd when I click the resolve the conversation and then the comment disapeared.

@wxiaoguang
Copy link
Contributor

But the experience is very odd when I click the resolve the conversation and then the comment disapeared.

Even if it doesn't disappear by clicking "resolve the conversation", it would also disappear when you refresh the page.

Indeed it is a design problem. Maybe it should tell the user that "The outdated comment won't show" on the UI.

@wxiaoguang
Copy link
Contributor

-> Improve user experience for outdated comments #29050

wxiaoguang added a commit that referenced this issue Feb 8, 2024
Try to improve #28949

1. Make `ctx.Data["ShowOutdatedComments"] = true` by default: it brings
consistent user experience, and sometimes the "outdated (source
changed)" comments are still valuable.
2. Show a friendly message if the comment won't show, then the end users
won't fell that "the comment disappears" (it is the special case when
`ShowOutdatedComments = false`)
GiteaBot pushed a commit to GiteaBot/gitea that referenced this issue Feb 8, 2024
Try to improve go-gitea#28949

1. Make `ctx.Data["ShowOutdatedComments"] = true` by default: it brings
consistent user experience, and sometimes the "outdated (source
changed)" comments are still valuable.
2. Show a friendly message if the comment won't show, then the end users
won't fell that "the comment disappears" (it is the special case when
`ShowOutdatedComments = false`)
wxiaoguang added a commit that referenced this issue Feb 8, 2024
Backport #29050 by wxiaoguang

Try to improve #28949

1. Make `ctx.Data["ShowOutdatedComments"] = true` by default: it brings
consistent user experience, and sometimes the "outdated (source
changed)" comments are still valuable.
2. Show a friendly message if the comment won't show, then the end users
won't fell that "the comment disappears" (it is the special case when
`ShowOutdatedComments = false`)

Co-authored-by: wxiaoguang <[email protected]>
@lunny lunny added this to the 1.21.6 milestone Feb 8, 2024
silverwind pushed a commit to silverwind/gitea that referenced this issue Feb 20, 2024
Try to improve go-gitea#28949

1. Make `ctx.Data["ShowOutdatedComments"] = true` by default: it brings
consistent user experience, and sometimes the "outdated (source
changed)" comments are still valuable.
2. Show a friendly message if the comment won't show, then the end users
won't fell that "the comment disappears" (it is the special case when
`ShowOutdatedComments = false`)
Copy link

Automatically locked because of our CONTRIBUTING guidelines

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants