Skip to content

Conversation

hojberg
Copy link
Contributor

@hojberg hojberg commented Oct 10, 2025

The API now sends back diffs on a line by line basis with spacers such
that left and right side diffs can be aligned. Update the JSON decoders
to support this new format and render the UI appropriately.

Also add line numbers and change indicators in a diff gutter as well as
line spacer patterns.


Depends on unisonweb/unison#5925

Copy link

netlify bot commented Oct 10, 2025

Deploy Preview for unison-share-ui ready!

Name Link
🔨 Latest commit 6369b5a
🔍 Latest deploy log https://app.netlify.com/projects/unison-share-ui/deploys/68f118828308570008da03c0
😎 Deploy Preview https://deploy-preview-118.staging.share.unison-lang.org
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 59 (🔴 down 6 from production)
Accessibility: 93 (no change from production)
Best Practices: 100 (🟢 up 8 from production)
SEO: 75 (no change from production)
PWA: 80 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@hojberg hojberg force-pushed the diff-alignment-integration branch 2 times, most recently from bb80869 to 2e5c0a2 Compare October 16, 2025 16:02
The API now sends back diffs on a line by line basis with spacers such
that left and right side diffs can be aligned. Update the JSON decoders
to support this new format and render the UI appropriately.

Also add line numbers and change indicators in a diff gutter as well as
line spacer patterns.
@hojberg hojberg force-pushed the diff-alignment-integration branch from 2e5c0a2 to 6369b5a Compare October 16, 2025 16:08
@hojberg hojberg marked this pull request as ready for review October 16, 2025 16:08
@hojberg hojberg requested a review from ChrisPenner October 16, 2025 16:08
@hojberg hojberg merged commit 006da1d into main Oct 16, 2025
6 checks passed
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