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

Improve counts on record page table of contents #2470

Open
jrchudy opened this issue May 24, 2024 · 0 comments
Open

Improve counts on record page table of contents #2470

jrchudy opened this issue May 24, 2024 · 0 comments
Labels
discussion required requires a discussion before moving forward record

Comments

@jrchudy
Copy link
Member

jrchudy commented May 24, 2024

The count on record page for related tables in the left panel (table of contents) represents the number of records that are visible for the loaded page of results and a + if there are more rows. We should improve this to update as the user has seen more rows from the related table.

For instance, say we have 31 rows for a related table and the page size for the related table is 10. The following is how the count should be represented as the user pages through that related table:

  • When the record page loads, related table is on "page 1" and count is shown as 10+
  • User pages the related table using next and now sees "page 2", count should be shown as 20+
  • User pages again using next and sees "page 3", count should be shown as 30+
  • User pages once more and sees the last page, count should be shown as 31

If the user pages back using previous, the count should remain the same since we have already seen that previous page.

If at any point, the content of the related table is fetched again and changes, this count state should be reset. This might only occur if rows are deleted by someone else while the user is looking at this page.

@RFSH RFSH added the record label Aug 14, 2024
@jrchudy jrchudy added the discussion required requires a discussion before moving forward label Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion required requires a discussion before moving forward record
Projects
None yet
Development

No branches or pull requests

2 participants