fix(svg-editor): show spec-violation banner when SVG lacks width/height#127
Conversation
Instead of silently backfilling dimensions, display a centered red banner telling the user to ask the AI to follow shared-standards.md §4. The banner is per-slide: removed on navigation, re-shown only when the incoming slide is missing width/height.
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
|
Thanks for the contribution. I think the direction here is right: live preview should not silently backfill root SVG dimensions on the frontend, because that would hide upstream SVG output that does not follow shared-standards.md §4. Surfacing the missing width/height as a spec violation is a better behavior than continuing to fail silently. I made one small follow-up fix before merging: the original implementation only checked for a missing width attribute, so it missed the case where height was absent. The merged version now triggers the warning when either width or height is missing. |
Closes #115. Surface spec violations with a red banner instead of silent blank preview. No backfill - upstream spec stays authoritative.