Repository navigation
fix(dashboard): keep the space in import rule summaries - #476
Conversation
Vue drops whitespace that opens an element, so the import preview read "Created (2):Too many lines". Put the separator inside the interpolation.
|
Architecture Data flow No data-flow sequence changed in this PR. Review ✨ Nothing flagged · reviewed Tip Open a diagram on the canvas, then press W or click play to walk through the change one step at a time 🪧 More tips
Thanks for using PR Lens! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out on X, Bluesky, Mastodon, Reddit or LinkedIn |
🤖 REVIEW PAUSED
This pull request is from an outside contributor. Add the
Only Wolfstar's click starts work. It adds the |
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes: the full single-commit diff (2 files).
- Rule summary spacing:
Transfer.vuemoves the separator into the interpolation ({{ " " + line.names }}). Vue strips a whitespace-only text node at the start of an element's children, so the old literal space was dropped. All four rule lines (created, edited, moved, deleted) share this one template, so the fix covers each of them. - Test tightened:
Transfer.spec.tsnow asserts"Created (1): Fresh"and"Deleted (1): Old". The old assertions checked label and names separately, so they could not catch the missing gap.
claude-sonnet-5-5 | 𝕏

Stacked on #451.
🔗 Linked issue
Related to #451.
🧭 Context
Vue drops whitespace that opens an element, so the import preview printed
Created (2):Too many lineswith no gap between the label and the rule names.📚 Description
The separator now sits inside the interpolation, so every rule summary (created, edited, moved, deleted) keeps it.


Confidence Score: 5/5
Safe to merge. Browser checks confirm that all four import summary categories retain the separator and preserve the rule names.Summary
Keeps a space between each import summary label and its rule names.
Reviews (1) · Last reviewed commit: "fix(settings): keep the space in import ..." · Reviewed by Greptile