refactor(ui): narrow local style forwarding and preview geometry - #8144
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
|
Before/after visual checks at All 18 light/dark states matched in computed geometry, typography, colours, borders and shadows. These are isolated source fixtures, not authenticated end-to-end flows: real markdown renderers use the installed Streamdown pipeline; secret fields use the actual component; dropdown fixtures use the actual anchor JSX in EMCN popovers with sample rows; PPTX fixtures use the actual wrapper methods with sample slide content. |
|


















Markdown cells and internal input/dropdown wrappers accept broader inline styles than their callers need. Restrict them to the properties actually used, and replace the PPTX preview wrappers’ interpolated CSS with explicit property assignments.
top,left,zIndex) and discard unused secret-field inline styles.No intended visual change. All 18 light/dark before-and-after fixture states matched in computed styling and geometry. All 18 before/after comparisons are attached here; they exercise the actual affected renderers/anchor code with local sample content, not authenticated end-to-end flows.
Validation: all 26 workspace type-check tasks, import-specifier check, API validation and changed-file lint pass; 63 relevant tests pass. Two existing chat timing assertions failed during the initial concurrent run and passed on the isolated rerun. The local design diff checker reports no new findings. Audit tooling and screenshots are outside the product diff.
GitHub CI passes, including the app build and PostgreSQL integration checks. Greptile and Cubic both report 5/5 on the current head.