Skip to content

Conversation

@sinisaos
Copy link
Member

Related to #463

@codecov-commenter
Copy link

codecov-commenter commented Dec 19, 2025

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.04%. Comparing base (1157c12) to head (0d90d2d).
⚠️ Report is 50 commits behind head on master.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #468      +/-   ##
==========================================
+ Coverage   93.42%   94.04%   +0.62%     
==========================================
  Files           5        5              
  Lines         365      403      +38     
==========================================
+ Hits          341      379      +38     
  Misses         24       24              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Member

@dantownsend dantownsend left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 👍

Can you use text-align: center on the input box?

Also, we need to detect if users click the up and down buttons on the input box - at the moment, it doesn't seem to move to the next/previous page.

Screenshot 2025-12-19 at 17 52 18

Comment on lines 398 to 404
<p id="result_count">
{{ $t("Showing") }} {{ rows.length }}
{{ $t("of") }}
{{ rowCount }}
{{ $t("result(s)") }}
</p>

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd keep this - it is sometimes useful to see the number of results.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

"For timestamps which are timezone aware, they will be displayed in this timezone by default.": "For timestamps which are timezone aware, they will be displayed in this timezone by default.",
"Form submitted": "Form submitted",
"Forms": "Forms",
"Go to page": "Go to page",
Copy link

@abhishek-compro abhishek-compro Dec 20, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not sure how it works in this project, but in my opinion translations shouldn't be removed.

Copy link
Member Author

@sinisaos sinisaos Dec 21, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we need it anymore because in this PR we don't use HTML select for pagination.

https://github.com/piccolo-orm/piccolo_admin/pull/468/files#diff-829d8edbf9a2076e6c9f6ddf97c687500222c30a38e068cfab593ba86cd1f11eL15-R21

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants