Skip to content

Refactor components to use Base UI and improve documentation - #48

Merged
Barata-Ribeiro merged 5 commits into
mainfrom
implement-minor-fixes
Aug 8, 2026
Merged

Refactor components to use Base UI and improve documentation#48
Barata-Ribeiro merged 5 commits into
mainfrom
implement-minor-fixes

Conversation

@Barata-Ribeiro

Copy link
Copy Markdown
Owner

This pull request updates the UI primitives used throughout the app from Radix UI to Base UI, refactors calendar popover and portal code to align with the new primitives, and updates related documentation and tests. It also improves accessibility and consistency in button and popover handling, and fixes minor UI details.

UI Primitives Migration and Documentation Updates

  • Replaced references to Radix UI with Base UI in the documentation (README.md, about.tsx) and updated descriptions and highlights accordingly. [1] [2] [3] [4]
  • Updated test comments to reference Base UI instead of Radix UI, and adjusted selectors for popover state to match Base UI conventions. [1] [2] [3] [4] [5] [6] [7]

Component Refactoring for Base UI

  • Refactored calendar popover triggers in date-add-subtract.tsx and date-difference.tsx to use buttonVariants for consistent styling and to match the new UI primitive's API. [1] [2] [3] [4]
  • Updated the Calendar component to require a buttonVariant prop and fixed className handling for range selection to match Base UI's requirements. [1] [2]

Portal Implementation Update

  • Rewrote the Portal component to use Base UI's useRender and removed the dependency on Radix's Slot, improving compatibility and future extensibility. [1] [2]

Accessibility and UX Improvements

  • Refactored the download button in base64-to-image.tsx to be a styled <Link> with correct accessibility attributes, ensuring keyboard and screen reader accessibility. [1] [2]
  • Improved test assertions for clarity and reliability, e.g., using .toHaveLength instead of .toBe for alert count checks.

These changes collectively modernize the codebase, improve accessibility, and lay groundwork for easier future UI updates.

@Barata-Ribeiro Barata-Ribeiro self-assigned this Aug 7, 2026
@Barata-Ribeiro Barata-Ribeiro added bug Something isn't working chore Handle config, moving etc. labels Aug 7, 2026
@vercel

vercel Bot commented Aug 7, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
utilities-webapp Ready Ready Preview Aug 7, 2026 4:28pm

@Barata-Ribeiro
Barata-Ribeiro merged commit 09884f5 into main Aug 8, 2026
3 checks passed
@Barata-Ribeiro
Barata-Ribeiro deleted the implement-minor-fixes branch August 8, 2026 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working chore Handle config, moving etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant