Skip to content

[Draft] fix: ui virtualization and test infrastructure#245

Draft
pixellos wants to merge 7 commits intoNeuralNomadsAI:devfrom
pixellos:fix/ui-virtualization-and-tests
Draft

[Draft] fix: ui virtualization and test infrastructure#245
pixellos wants to merge 7 commits intoNeuralNomadsAI:devfrom
pixellos:fix/ui-virtualization-and-tests

Conversation

@pixellos
Copy link
Copy Markdown
Contributor

Summary

  • Finalizes virtualization implementation in MessageTimeline.
  • Adds Playwright component tests for MessageTimeline and SessionView.
  • Cleans up SessionView and updates preferences store.

@pixellos pixellos changed the title fix: ui virtualization and test infrastructure [Draft] fix: ui virtualization and test infrastructure Mar 24, 2026
@github-actions
Copy link
Copy Markdown

PR builds are available as GitHub Actions artifacts:

https://github.com/NeuralNomadsAI/CodeNomad/actions/runs/23517051274

Artifacts expire in 7 days.
Artifacts: (none found on this run)

@pixellos
Copy link
Copy Markdown
Contributor Author

It seems i also need to bump up node to recover pipelines - will have a look tomorrow

@github-actions
Copy link
Copy Markdown

PR builds are available as GitHub Actions artifacts:

https://github.com/NeuralNomadsAI/CodeNomad/actions/runs/23517194227

Artifacts expire in 7 days.
Artifacts: (none found on this run)

@github-actions
Copy link
Copy Markdown

PR builds are available as GitHub Actions artifacts:

https://github.com/NeuralNomadsAI/CodeNomad/actions/runs/23517457884

Artifacts expire in 7 days.
Artifacts: (none found on this run)

@pixellos pixellos force-pushed the fix/ui-virtualization-and-tests branch from b3e7dc2 to f4130e5 Compare March 24, 2026 23:40
@github-actions
Copy link
Copy Markdown

PR builds are available as GitHub Actions artifacts:

https://github.com/NeuralNomadsAI/CodeNomad/actions/runs/23517551273

Artifacts expire in 7 days.
Artifacts: (none found on this run)

@github-actions
Copy link
Copy Markdown

PR builds are available as GitHub Actions artifacts:

https://github.com/NeuralNomadsAI/CodeNomad/actions/runs/23517646004

Artifacts expire in 7 days.
Artifacts: (none found on this run)

@github-actions
Copy link
Copy Markdown

PR builds are available as GitHub Actions artifacts:

https://github.com/NeuralNomadsAI/CodeNomad/actions/runs/23530870609

Artifacts expire in 7 days.
Artifacts: (none found on this run)

@github-actions
Copy link
Copy Markdown

PR builds are available as GitHub Actions artifacts:

https://github.com/NeuralNomadsAI/CodeNomad/actions/runs/23531423876

Artifacts expire in 7 days.
Artifacts: (none found on this run)

@shantur shantur marked this pull request as draft March 25, 2026 09:11
@shantur
Copy link
Copy Markdown
Collaborator

shantur commented Mar 25, 2026

@pixellos - Thanks for the PR
The scope of this PR is a lot bigger and implementing unrelated things. It will not be possible to review the changes with multiple features. Can we have PRs with one change each please.
Thanks

@pixellos
Copy link
Copy Markdown
Contributor Author

pixellos commented Mar 25, 2026 via email

@shantur
Copy link
Copy Markdown
Collaborator

shantur commented Mar 25, 2026

100%

  • Adding tests is always helpful
  • Virtualization for timeline isn't that important TBH as there aren't many DOM elements in there. The main chat has too many DOM nodes because of the Markdown and syntax highlighting. If you want to, it will surely be better.

@pixellos
Copy link
Copy Markdown
Contributor Author

pixellos commented Mar 25, 2026 via email

@shantur
Copy link
Copy Markdown
Collaborator

shantur commented Mar 25, 2026

That should work, thanks.

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.

2 participants