Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add form builder icons #155

Merged
merged 3 commits into from
Nov 25, 2024
Merged

Add form builder icons #155

merged 3 commits into from
Nov 25, 2024

Conversation

skrobek
Copy link
Collaborator

@skrobek skrobek commented Nov 25, 2024

PR Type

enhancement


Description

  • Added new icons for form builder functionality in src/components/ui/icon/types.ts, enhancing the icon set with RiQuoteText, RiTextSnippet, RiRadioButtonLine, RiCheckboxLine, RiEqualizer2Line, RiArticleLine, and RiBillLine.
  • Updated the package version in package.json from 0.12.1 to 0.12.2 to reflect the new changes.

Changes walkthrough 📝

Relevant files
Enhancement
types.ts
Add new form builder icons to the ICONS export                     

src/components/ui/icon/types.ts

  • Added new icons to the import list from @remixicon/react.
  • Updated the ICONS export to include new icons.
  • +16/-2   
    Configuration changes
    package.json
    Bump package version to 0.12.2                                                     

    package.json

    • Updated the package version from 0.12.1 to 0.12.2.
    +1/-1     

    💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

    Copy link

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Duplicate Code
    The same set of icons are added twice in the file, which might be a mistake or oversight. This could lead to confusion and maintenance issues.

    Duplicate Code
    The same set of icons are added twice in the file, which might be a mistake or oversight. This could lead to confusion and maintenance issues.

    Copy link

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @skrobek skrobek merged commit 9e8e778 into main Nov 25, 2024
    1 check passed
    @skrobek skrobek deleted the icons-form-builder branch December 17, 2024 09:05
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    1 participant