Skip to content

feat(KLEF-135): Revamp components - #1

Closed
Chronyyx wants to merge 3 commits into
mainfrom
feat/KLEF-135-Revamp_Components
Closed

feat(KLEF-135): Revamp components#1
Chronyyx wants to merge 3 commits into
mainfrom
feat/KLEF-135-Revamp_Components

Conversation

@Chronyyx

Copy link
Copy Markdown
Contributor

Pull Request

Summary

This PR revamps the components-plugin showcase page to integrate the latest UI patterns. It delegates toast management to the unified @kleffio/ui toaster and introduces new examples for our updated collapsible components. Additionally, the plugin route has been cleanly nested under /p/components.

Related Issues

Closes #KLEF-135


Changes

What's Included

  • Migrated the toast import from sonner directly to @kleffio/ui to utilize the shared toast UI.
  • Rewrote the collapsible demos to use the new CollapsibleTriggerIcon and added a new FolderCollapsible variant.
  • Updated the Section wrapper to support contextual descriptions and added helpful descriptions to the Alerts and Toasts sections.
  • Updated the plugin route namespace from /components to /p/components in the plugin definition (src/index.ts).

What's Not Included

  • No new UI components were added to @kleffio/ui in this PR (it only consumes them).
  • No changes to the underlying platform API.

Testing

How Was This Tested?

  • Tested locally by navigating to the new /p/components route.
  • Verified that firing a toast correctly triggers the UI without double-rendering locally.
  • Confirmed the expanded/collapsed state and animations of the new FolderCollapsible component.

Test Coverage

  • Unit tests added or updated
  • Integration tests added or updated
  • Manually tested end-to-end

Breaking Changes

Does this PR introduce breaking changes?

  • Yes
  • No

If yes, describe the impact and any required migration steps:


Security Considerations

  • This PR affects authentication or authorization logic
  • This PR touches secrets, tokens, or environment variables
  • This PR affects infrastructure, deployment pipelines, or network configuration

If any box is checked, describe the security impact and any mitigations applied:


Documentation

Does this PR require documentation updates?

  • Yes
  • No

If yes, check all that apply:

  • README.md
  • ARCHITECTURE.md
  • API reference
  • Architecture diagrams (C4 / DDD)
  • /docs

UI/UX (If Applicable)

  • Includes new components or views
  • Changes styles or theme tokens
  • Responsive behavior verified
  • Requires design approval

Screenshots or recordings:
(Feel free to drop a screenshot of the new FolderCollapsible component here)


Pre-Merge Checklist

  • PR title follows semantic format (feat:, fix:, chore:, docs:, refactor:, test:)
  • All CI checks passing
  • Code follows project style guidelines
  • No debug logs or commented-out code left in
  • Dependencies reviewed (no unnecessary additions)
  • No sensitive information included

Reviewer Notes

This PR primarily aligns the component showcase with the most recent updates to @kleffio/ui.

@Chronyyx Chronyyx self-assigned this Apr 16, 2026
@Chronyyx Chronyyx changed the title feat: add components to showcase page and export index for components… feat(KLEF-135): Revamp components Apr 16, 2026
@Chronyyx Chronyyx added the enhancement Enhancement to existing feature label Apr 16, 2026
@Chronyyx Chronyyx closed this May 6, 2026
@Chronyyx
Chronyyx deleted the feat/KLEF-135-Revamp_Components branch May 6, 2026 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Enhancement to existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants