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

Viewed files counter is reset when performing a review #28239

Closed
saneks222 opened this issue Nov 27, 2023 · 1 comment
Closed

Viewed files counter is reset when performing a review #28239

saneks222 opened this issue Nov 27, 2023 · 1 comment
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented type/bug
Milestone

Comments

@saneks222
Copy link

Description

Familiarized with the issue
#26854 ,
#20681

Looks like the problem persists

Gitea Version: 1.21.0

Steps to Replay:

Create a repo
Create a PR with a large number of modified files
Start viewing the PR, marking each viewed file as "viewed" using the checkboxes
Once the first 100 files are marked as viewed, click the "load more" button at the end of the page to load the remaining files

Expected behavior:

100 files are marked as viewed
Viewed files counter at the top of the page displays 100

Actual behavior:

100 files marked as viewed
the viewed files counter at the top of the page displays 0

image

Gitea Version

1.21.0

Can you reproduce the bug on the Gitea demo site?

Yes

Log Gist

No response

Screenshots

No response

Git Version

1.21.0

Operating System

No response

How are you running Gitea?

Official docker image.

Database

PostgreSQL

@sillyguodong sillyguodong added the issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented label Feb 20, 2024
@lunny lunny added this to the 1.21.8 milestone Feb 27, 2024
6543 pushed a commit that referenced this issue Feb 28, 2024
issue : #28239

The counter number script uses the 'checkbox' attribute to determine
whether an item is selected or not.

However, the input event only increments the counter value, and when
more items are displayed, it does not update all previously loaded
items.

As a result, the display becomes incorrect because it triggers the
update counter script, but checkboxes that are selected without the
'checked' attribute are not counted
GiteaBot pushed a commit to GiteaBot/gitea that referenced this issue Feb 28, 2024
…a#29448)

issue : go-gitea#28239

The counter number script uses the 'checkbox' attribute to determine
whether an item is selected or not.

However, the input event only increments the counter value, and when
more items are displayed, it does not update all previously loaded
items.

As a result, the display becomes incorrect because it triggers the
update counter script, but checkboxes that are selected without the
'checked' attribute are not counted
silverwind pushed a commit that referenced this issue Feb 28, 2024
…#29478)

Backport #29448 by @charles7668

issue : #28239

The counter number script uses the 'checkbox' attribute to determine
whether an item is selected or not.

However, the input event only increments the counter value, and when
more items are displayed, it does not update all previously loaded
items.

As a result, the display becomes incorrect because it triggers the
update counter script, but checkboxes that are selected without the
'checked' attribute are not counted

Co-authored-by: charles <30816317+charles7668@users.noreply.github.com>
@sillyguodong
Copy link
Contributor

closed by #29448

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented type/bug
Projects
None yet
Development

No branches or pull requests

3 participants