Skip to content

feat: added the button component#27

Closed
Aarif5435 wants to merge 1 commit intoSigNoz:mainfrom
Aarif5435:feat/add-button-component
Closed

feat: added the button component#27
Aarif5435 wants to merge 1 commit intoSigNoz:mainfrom
Aarif5435:feat/add-button-component

Conversation

@Aarif5435
Copy link

Feature: Button Component Implementation Based on Figma Design

Description

This PR introduces a new Button component built based on the Figma design provided in issue #13. The component has been added to Storybook for ease of visualization and testing. The design follows the guidelines outlined in the issue, including support for various states and styling derived from design tokens.

Key Features

  • Basic UI Implementation: Button layout and style closely mirror the Figma design.
  • Prop Support: Props introduced for better flexibility and adherence to Figma specifications:
    • variant: Defines the button style (e.g., primary, secondary, warning, etc.).
    • size: Supports different sizes (xs, sm, md, lg).
    • asChild: Allows rendering the button as another component (e.g., Link).
    • theme: Supports light and dark themes.
  • Dynamic Styling: Uses Tailwind CSS for consistent, responsive styles and design-token-based color variables.
  • State Management:
    • Normal: Default state of the button.
    • Hover: Adds hover effects per Figma design.
    • Active: Highlights the active button state for better UX.

Screenshorts

image

@Aarif5435 Aarif5435 force-pushed the feat/add-button-component branch 8 times, most recently from c90a24a to c29b417 Compare November 22, 2024 12:36
@Aarif5435 Aarif5435 force-pushed the feat/add-button-component branch from c29b417 to 0eeb3a7 Compare November 22, 2024 12:46
@ahmadshaheer
Copy link
Contributor

closing in favor of #48

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