Skip to content

Refactoring components and cleanup#13

Merged
eugenebelov merged 12 commits into
masterfrom
refactoring-components-and-cleanup
Aug 26, 2025
Merged

Refactoring components and cleanup#13
eugenebelov merged 12 commits into
masterfrom
refactoring-components-and-cleanup

Conversation

@eugenebelov

@eugenebelov eugenebelov commented Aug 21, 2025

Copy link
Copy Markdown
Member
  • Resolves: #

Summary

  • Added two new components: Address and HashLink
  • Fix animation and refactored inside Copy.
  • Refactored Tooltip component. renamed property title to tooltipContent

Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • 🔧 Component enhancement
  • 📚 Documentation update
  • 🎨 Style/UI changes
  • ⚙️ Misc

Component(s) Affected

  • New component: Address, HashLink
  • Modified component: ComponentName
  • Removed component: ComponentName

Changes Made

Testing

  • Unit tests added/updated
  • Storybook stories added/updated
  • Manual testing completed
  • Accessibility testing performed
  • Cross-browser testing (if applicable)
  • Responsive design testing (if applicable)

Storybook

  • New stories created for new components
  • Existing stories updated to reflect changes
  • All component variants documented
  • Props table is accurate and complete
  • Examples demonstrate proper usage

Breaking Changes

  • Tooltip prop title became tooltipContent

Screenshots/Videos

TODO

  • ...

Checklist

  • Code is properly formatted (Prettier/ESLint)
  • All commits are signed
  • TypeScript types are properly defined
  • Components follow design system guidelines
  • Accessibility requirements met (WCAG compliance)
  • Bundle size impact considered
  • Tests included/updated or not needed
  • Storybook stories included/updated or not needed
  • Documentation updated or not required
  • No console errors or warnings
  • Backward compatibility maintained (or breaking changes documented)

@eugenebelov eugenebelov self-assigned this Aug 21, 2025
@eugenebelov eugenebelov added the enhancement New feature or request label Aug 21, 2025

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors several components and adds new functionality to improve the design system. The main changes include adding two new display components (Address and HashLink), refactoring the Tooltip component with a renamed prop, enhancing the Copy component with improved animations, and updating the Avatar component with new size options.

  • Added new Address and HashLink components for displaying account information with proper formatting
  • Refactored Tooltip component by renaming the title prop to tooltipContent (breaking change)
  • Enhanced Copy component with fade-in/out animations and improved styling

Reviewed Changes

Copilot reviewed 17 out of 18 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/lib/utils/formatters.ts Added utility function for truncating CSPR names
src/lib/components/tooltip/tooltip.tsx Renamed title prop to tooltipContent and updated styling
src/lib/components/tooltip/tooltip.stories.tsx Updated stories to use new prop name and improved examples
src/lib/components/copy/copy.tsx Refactored with new animations, styling, and improved user interaction
src/lib/components/avatar/avatar.tsx Added new size options and transparent background support
src/lib/components/address/address.tsx New component for displaying account addresses with avatars and formatting
src/lib/components/hash-link/hash-link.tsx New component for displaying hash links with copy functionality
Multiple component files Updated to use new tooltipContent prop instead of title
package.json Moved prettier from dependencies to devDependencies

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread src/lib/utils/formatters.ts Outdated
Comment thread src/lib/components/copy/copy.tsx Outdated
Comment thread src/lib/components/address/address.tsx Outdated
Comment thread src/lib/components/address/address.tsx Outdated
eugenebelov and others added 2 commits August 26, 2025 11:24
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

@eugenebelov eugenebelov left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@victoriari how do you think, should we keep truncateCSPRName here in formatters or move somewhere else?
if we may keep it, maybe I can add shortenCsprName function here as well?

@eugenebelov eugenebelov merged commit df23d88 into master Aug 26, 2025
5 checks passed
@eugenebelov eugenebelov deleted the refactoring-components-and-cleanup branch August 26, 2025 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants