Retake the README screenshots on the v1.1.0 UI - #8
Merged
Conversation
Uppercase the drop-zone label through text-transform rather than literal caps, so case stays in the stylesheet like the app's other uppercase runs. Tracked at .06em to sit alongside .card-title, the only other large-size uppercase in the UI. Rename the sample link from "Use sample" to "Load a sample". Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
The four screenshots still showed the pre-redesign interface: none of them had the new masthead or the readout strip. Retaken against the same sample files so the existing alt text still describes what is on screen. Two things changed beyond the styling: - The Claims table was clipped on the right in the old capture, cutting off the claim total column. It now fits, so the alt text gains that column. - The old Tree capture reported cob_loops.edi as 51 segments / 1.3 KB. The file is 44 segments / 1,244 bytes and has never been edited; those were the figures for patient_and_cob.edi, loaded before it. The readout went stale on load and the filename did not. Fixed by the load path work in #5, and the new capture reads correctly. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes the last item left over from the v1.1.0 UI redesign: the four README
screenshots still showed the pre-redesign interface. Nothing in the repo
pictured the new masthead or the readout strip, so anyone arriving at the
README saw an app that does not match what they would download.
Screenshots
Retaken against the same sample files as the originals, so the existing alt
text still describes what is on screen:
document-tab.pngdeid/patient_and_cob.editree-tab.pngtree/cob_loops.ediHIcomposite expandedclaims-tab.pngdeid/patient_and_cob.edilimited-data-set.pngdeid/repeat_patient.ediAll four are dark theme, mask off, and within a few pixels of each other so
they stack consistently.
Two things that were not just styling
The Claims table was clipped. The old capture cut off the claim total
column mid-word. It now fits, so the alt text gains that column — it had
been accurate only because the column was invisible.
The old Tree capture had a bug in it. It reported
cob_loops.edias51 segments / 1.3 KB. The file is 44 segments and 1,244 bytes, and
git logshows it has not been touched since it was committed. Those were
patient_and_cob.edi's figures — the file loaded before it. The readoutwent stale on load while the filename updated, which is the same load path
that #5 fixed and that
render.mjswas built to cover. The new capturereads
44 / 1.2 KB.Copy
Two small changes to the source-interchange panel, included here because the
screenshots show them:
text-transformrather thanliteral caps, so case lives in the stylesheet like the app's other
uppercase runs. Tracked at
.06em, matching.card-title.Use samplereadsLoad a sample.Verification
node web/tests/all.mjs— all 9 suites pass. No suite references eitherstring, and no behaviour changed.
🤖 Generated with Claude Code