Skip to content

feat: improve pagination block#2347

Open
Koc wants to merge 2 commits intomainfrom
feature/improve-pagination-block
Open

feat: improve pagination block#2347
Koc wants to merge 2 commits intomainfrom
feature/improve-pagination-block

Conversation

@Koc
Copy link
Copy Markdown
Contributor

@Koc Koc commented Mar 2, 2026

Current pagination block has few drawbacks:

  • it is not possible to tune how many items per page to display
  • block located in the very bottom of the page on the left side. So it is very inconvenient for wide table with multiple rows (you need to scroll all the time)
  • pagination not visible in the embedded views (closes Embedded View has only data from first page #2073)
🔍 Screenshoots

Initial view
image

Large table scrolled to the bottom
image

Large table scrolled to the right with few rows selected
image

Embedded table into collective
image

So, this PR aims to fix all of them.

⚠️ Styles are not perfect (I'm not CSS master, sorry 🙈) but it's better than it now. Feel free to improve anything or just pushing directly into the branch

@Aveyron-RetD
Copy link
Copy Markdown

Maybe this could solve ? #2073

@Koc Koc force-pushed the feature/improve-pagination-block branch from db76734 to c293e93 Compare March 15, 2026 22:08
@Koc Koc force-pushed the feature/improve-pagination-block branch 3 times, most recently from b596f23 to 2aa6d1e Compare March 26, 2026 10:59
@Koc Koc marked this pull request as ready for review March 26, 2026 11:02
@Koc Koc requested review from blizzz and enjeck as code owners March 26, 2026 11:02
@Koc
Copy link
Copy Markdown
Contributor Author

Koc commented Mar 26, 2026

@Aveyron-RetD sorry, but I have no idea what is "smart picker", "full content mode" and other stuff from the referenced ticket 😄 .

Feel free to test mentioned bug with my branch and/or provide video/screenshoots for the better understanding

@Aveyron-RetD
Copy link
Copy Markdown

The smart picker is the way to have access to another app inside a .md document or Richtextfield in Nextcloud.
Fun fact you can insert a table within a rich text field of a cell in the table app).

You have access to the smart picker with the slash bar / .

Tables from the Tables app can be seen and manipulated directly in any .md files when they are inserted with the smart picker.

But the Tables app is for the moment lets say not very usable within the other document because they are too many limitations or bugs.

Like for pagination. The view is limited to the 100 first row because the bottom of the page does not contain the pagination bar. There could be also another mechanism at play in here because the search as well do not look further than those 100 rows.

image

Tested with a table with 1000 row.
MOCK_DATA.csv

And in here you cannot get to the next page...
image

@Koc Koc force-pushed the feature/improve-pagination-block branch 5 times, most recently from b9dccf8 to 94bc082 Compare March 31, 2026 15:19
@Koc
Copy link
Copy Markdown
Contributor Author

Koc commented Mar 31, 2026

@Aveyron-RetD I guess that it should help. This is how it looks with changes from PR
image

@enjeck @blizzz please check

@Koc Koc force-pushed the feature/improve-pagination-block branch from 94bc082 to ad8ac39 Compare March 31, 2026 15:19
@Aveyron-RetD
Copy link
Copy Markdown

Thanks for all your contribution @Koc !!

Koc added 2 commits April 1, 2026 15:26
Signed-off-by: Kostiantyn Miakshyn <molodchick@gmail.com>
Signed-off-by: Kostiantyn Miakshyn <molodchick@gmail.com>
@Koc Koc force-pushed the feature/improve-pagination-block branch from ad8ac39 to 9fbe5c9 Compare April 1, 2026 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Embedded View has only data from first page

2 participants