Skip to content

fix(3433): jobs table sorting across pages and keep visible row data populated#1565

Open
yk634 wants to merge 2 commits intomasterfrom
job-list-view-sort
Open

fix(3433): jobs table sorting across pages and keep visible row data populated#1565
yk634 wants to merge 2 commits intomasterfrom
job-list-view-sort

Conversation

@yk634
Copy link
Contributor

@yk634 yk634 commented Mar 24, 2026

Context

Sorting in the jobs table was not fully consistent across the dataset.
In particular, START TIME sorting did not correctly include records outside the current page.
Also, when sorting changed which records were visible on the current page, row-dependent data such as HISTORY and DURATION could become empty because the loaded build data no longer matched the displayed rows.

Objective

This PR fixes jobs table sorting so that START TIME sorting works across the full filtered dataset and the displayed rows remain populated after sorting and pagination changes.

Intentional changes in this PR:

  • make START TIME sorting work across records outside the current page
  • keep jobs table sorting behavior consistent when switching between sortable headers
  • keep build reload targets aligned with the current sorted rows so visible rows stay populated after sorting
  • ignore stale build status responses that arrive after a newer reload starts

References

screwdriver-cd/screwdriver#3433

License

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

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.

1 participant