Skip to content

feat(fieldsets) - use base description#995

Merged
gabrielseco merged 1 commit intomainfrom
use-base-description-in-fieldsets
May 5, 2026
Merged

feat(fieldsets) - use base description#995
gabrielseco merged 1 commit intomainfrom
use-base-description-in-fieldsets

Conversation

@gabrielseco
Copy link
Copy Markdown
Collaborator

@gabrielseco gabrielseco commented May 4, 2026

Note

Low Risk
Low risk UI refactor that changes how fieldset descriptions are rendered/sanitized; potential risk is slight differences in HTML sanitization/rendering for existing descriptions.

Overview
Updates FieldSetField to render the fieldset description using the shared BaseFormDescription component instead of manually calling sanitizeHtml with dangerouslySetInnerHTML.

This centralizes description sanitization/styling and removes the direct sanitizeHtml dependency from FieldSetField.

Reviewed by Cursor Bugbot for commit f404daf. Bugbot is set up for automated code reviews on this repo. Configure here.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

📦 Bundle Size Report

Metric Current Previous Change Status
Total (gzip) 211.72 kB 211.72 kB -5 B (-0.0%) 🟢
Total (raw) 572.03 kB 572.06 kB -22 B (-0.0%) 🟢
CSS (gzip) 20.69 kB 20.69 kB 0 B (0%) 🟢
CSS (raw) 107.95 kB 107.95 kB 0 B (0%) 🟢

Size Limits

  • ✅ Total gzipped: 211.72 kB / 250 kB (84.7%)
  • ✅ Total raw: 572.03 kB / 600 kB (95.3%)
  • ✅ CSS gzipped: 20.69 kB / 25 kB (82.8%)

Largest Files (Top 5)

  1. chunk-SJZP5XKE.js - 13.93 kB (0 B (0%))
  2. styles.css - 10.35 kB (0 B (0%))
  3. index.css - 10.35 kB (0 B (0%))
  4. index.js - 6.07 kB (-3 B (-0.0%))
  5. chunk-TNQKP2XT.js - 5.89 kB (0 B (0%))
View All Files (313 total)
File Size (gzip) Change
chunk-SJZP5XKE.js 13.93 kB 0 B (0%)
styles.css 10.35 kB 0 B (0%)
index.css 10.35 kB 0 B (0%)
index.js 6.07 kB -3 B (-0.0%)
chunk-TNQKP2XT.js 5.89 kB 0 B (0%)
chunk-KXECMRUA.js 5.81 kB new
chunk-VFD5QMQA.js 4.55 kB new
chunk-WUKEIUCM.js 4.29 kB new
chunk-DPVXTQAB.js 4.06 kB 0 B (0%)
chunk-TQQ227SJ.js 3.73 kB 0 B (0%)

✅ Bundle size check passed

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

📊 Coverage Report

⚪ Coverage unchanged

Metric Current Previous Change Status
Lines 89.95% 89.95% 0%
Statements 89.62% 89.62% 0%
Functions 87.69% 87.69% 0%
Branches 80.01% 80.01% 0%

Detailed Breakdown

Lines Coverage
  • Covered: 3651 / 4059
  • Coverage: 89.95%
  • Change: 0% (0 lines)
Statements Coverage
  • Covered: 3712 / 4142
  • Coverage: 89.62%
  • Change: 0% (0 statements)
Functions Coverage
  • Covered: 997 / 1137
  • Coverage: 87.69%
  • Change: 0% (0 functions)
Branches Coverage
  • Covered: 2265 / 2831
  • Coverage: 80.01%
  • Change: 0% (0 branches)

✅ Coverage check passed

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

Deploy preview for remote-flows ready!

Project:remote-flows
Status: ✅  Deploy successful!
Preview URL:https://remote-flows-jav519df3-remotecom.vercel.app
Latest Commit:f404daf
Inspect:View deployment

Deployed with vercel-action

@gabrielseco gabrielseco merged commit c41ba9f into main May 5, 2026
10 checks passed
@gabrielseco gabrielseco deleted the use-base-description-in-fieldsets branch May 5, 2026 07:04
@gabrielseco gabrielseco mentioned this pull request May 8, 2026
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.

2 participants