Skip to content

feat(ui): allow button type to overwritten#1541

Merged
vlad-schur-external-sap merged 7 commits intomainfrom
vlad-button-type-override
Mar 16, 2026
Merged

feat(ui): allow button type to overwritten#1541
vlad-schur-external-sap merged 7 commits intomainfrom
vlad-button-type-override

Conversation

@vlad-schur-external-sap
Copy link
Copy Markdown
Contributor

@vlad-schur-external-sap vlad-schur-external-sap commented Mar 16, 2026

Summary

Allow <Button type /> to be overridden, provides flexibility for different use cases (e.g., using "submit", "reset", or "button"). It prevents unintended form submissions or lets developers control behavior without manually wrapping or replacing the component.

Changes Made

  • Added customizable type property for the button component.
  • Replaced deprecated LegacyRef with the new Ref type.
  • Updated hardcoded rem values to use predefined Tailwind utility classes (Button and AppShell).

Related Issues

Testing Instructions

  1. pnpm i
  2. pnpm TASK

Checklist

  • I have performed a self-review of my code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.
  • I have made corresponding changes to the documentation (if applicable).
  • My changes generate no new warnings or errors.
  • I have created a changeset for my changes.

PR Manifesto

Review the PR Manifesto for best practises.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Mar 16, 2026

🦋 Changeset detected

Latest commit: 50b889b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 9 packages
Name Type
@cloudoperators/juno-ui-components Patch
@cloudoperators/juno-app-carbon Patch
@cloudoperators/juno-app-doop Patch
@cloudoperators/juno-app-example Patch
@cloudoperators/juno-app-greenhouse Patch
@cloudoperators/juno-app-heureka Patch
@cloudoperators/juno-app-supernova Patch
@cloudoperators/juno-app-template Patch
@cloudoperators/juno-messages-provider Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vlad-schur-external-sap vlad-schur-external-sap marked this pull request as draft March 16, 2026 14:40
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 16, 2026

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-03-16 16:36 UTC

@vlad-schur-external-sap vlad-schur-external-sap marked this pull request as ready for review March 16, 2026 14:54
@vlad-schur-external-sap vlad-schur-external-sap added the ui-components All tasks related to juno-ui-components library label Mar 16, 2026
Copy link
Copy Markdown
Collaborator

@ArtieReus ArtieReus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great!! lgtm

Copy link
Copy Markdown
Collaborator

@andypf andypf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great Job!

@vlad-schur-external-sap vlad-schur-external-sap merged commit 11eabd6 into main Mar 16, 2026
23 checks passed
@vlad-schur-external-sap vlad-schur-external-sap deleted the vlad-button-type-override branch March 16, 2026 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ui-components All tasks related to juno-ui-components library

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants