Skip to content

fix(viewer): mount the compared elements again on a retry - #155

Open
skjnldsv wants to merge 1 commit into
mainfrom
fix/comparison-retry
Open

skjnldsv wants to merge 1 commit into
mainfrom
fix/comparison-retry

Conversation

@skjnldsv

Copy link
Copy Markdown
Contributor

"Try again" in a comparison kept spinning forever: the compared elements stay mounted behind the error, and their key ignored reloadKey, so nothing remounted. Side by side, the retry also only waited for one of the two.

Same fix makes a new version of the compared file show up, like in the single view. Follow-up to #153.

👾 This pull request was assisted by Claude Code, commits carry an Assisted-by trailer.

The elements of a comparison stay mounted, hidden, while an error
shows, and their keys did not follow reloadKey: a retry mounted
nothing, no element said it had loaded, and the spinner stayed. A new
version of the compared file was not shown either. Side by side, a
retry also waited for one element instead of two.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: John Molakvoæ <14975046+skjnldsv@users.noreply.github.com>
@skjnldsv skjnldsv added this to the 2.0.0 milestone Oct 20, 2026
@skjnldsv skjnldsv added type: bug 🐛 Something isn't working AI assisted 3. to review labels Oct 20, 2026
@skjnldsv skjnldsv self-assigned this Oct 20, 2026
@codecov

codecov Bot commented Oct 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.54%. Comparing base (df13a4f) to head (6876840).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #155   +/-   ##
=======================================
  Coverage   92.54%   92.54%           
=======================================
  Files          51       51           
  Lines        4187     4187           
  Branches      928      930    +2     
=======================================
  Hits         3875     3875           
  Misses        289      289           
  Partials       23       23           

☔ View full report in Codecov by Harness.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@skjnldsv
skjnldsv requested a review from mejo- October 20, 2026 08:10

@mejo- mejo- left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code changes look good.

This branch has not been deployed

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

Labels

3. to review AI assisted type: bug 🐛 Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants