fix(ui): don't show pagination warning on first page if all are displ…

…ayed (#11979)
argoproj · Oct 14, 2023 · 67c5fc9 · 67c5fc9
1 parent ac10548
commit 67c5fc9
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/ui/src/app/shared/components/pagination-panel.tsx b/ui/src/app/shared/components/pagination-panel.tsx
@@ -19,7 +19,8 @@ export function PaginationPanel(props: {pagination: Pagination; onChange: (pagin
                 }>
                 Next page <i className='fa fa-chevron-right' />
             </button>
-            {props.pagination.limit > 0 ? (
+            {/* if pagination is used, and we're either not on the first page, or are on the first page and have more records than the page limit */}
+            {props.pagination.limit > 0 && (props.pagination.offset || (!props.pagination.offset && props.numRecords >= props.pagination.limit)) ? (
                 <>
                     <WarningIcon /> Workflows cannot be globally sorted when paginated
                 </>