Skip to content

feat: add PencilIcon, TagIcon and ColorPicker (LAI-5067)#22

Merged
Mateus Veloso (mateusveloso) merged 1 commit intomainfrom
feat/LAI-5067-tag-icons-colorpicker
Mar 30, 2026
Merged

feat: add PencilIcon, TagIcon and ColorPicker (LAI-5067)#22
Mateus Veloso (mateusveloso) merged 1 commit intomainfrom
feat/LAI-5067-tag-icons-colorpicker

Conversation

@mateusveloso
Copy link
Copy Markdown
Contributor

@mateusveloso Mateus Veloso (mateusveloso) commented Mar 30, 2026

Summary

Adiciona componentes necessários para a feature de Etiquetas (LAI-5067):

  • PencilIcon — ícone de edição (lápis)
  • TagIcon — ícone de etiqueta
  • ColorPicker — componente de seleção de cor com hue slider, hex input e preset swatches (usa react-colorful)

Esses componentes estavam locais no casa-da-lais e precisam ser migrados pro DS conforme feedback do Vitor Leal (@vitorleal).

Test plan

  • Verificar que PencilIcon e TagIcon renderizam corretamente em stories
  • Verificar que ColorPicker funciona com seleção de cor, input hex e presets
  • Verificar que exports estão corretos via @lastro-co/design-system e @lastro-co/design-system/icons

Summary by CodeRabbit

Release Notes

  • New Features
    • Added a color picker with hex input and hue slider adjustment
    • Added Pencil and Tag icon components for UI customization

- Add PencilIcon and TagIcon to the icons library
- Add ColorPicker component with hex input, hue slider and preset swatches
- Add react-colorful dependency for the color picker

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 30, 2026

Walkthrough

Yo, listen up, fool! This pull request be addin' the react-colorful dependency to the package, sucka. Two fresh icon components—PencilIcon and TagIcon—got added to the icon suite. A brand new ColorPicker UI component be joinin' the party with color management, hex input validation, and a hue slider. All the barrel exports be updated to expose these new components, I tell ya!

Changes

Cohort / File(s) Summary
Dependencies
package.json
Added react-colorful (^5.6.1) dependency, fool!
Icon Components
src/components/icons/PencilIcon.tsx, src/components/icons/TagIcon.tsx, src/components/icons/index.ts
Two new icon components (PencilIcon, TagIcon) created with SVG paths and proper typing. Barrel exports updated to expose both icons, sucka!
ColorPicker Component
src/components/ui/ColorPicker/ColorPicker.tsx, src/components/ui/ColorPicker/index.ts, src/components/ui/index.ts
New ColorPicker component with state management for draft color, hex input validation (/^#[0-9a-fA-F]{0,6}$/), hue slider using HSL conversion, and popover interaction. Includes ColorPickerProps interface and barrel exports at multiple levels, I pity the fool who misses this!

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🎨 Colors be poppin', icons be droppin',
Mr. T's ColorPicker won't stop rockin',
Hue sliders spinnin', hex codes be ginnin',
This code so fine, fool, can't you see the winnin'!
I pity the fool who can't dig this vibe! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main changes: adding three new components (PencilIcon, TagIcon, ColorPicker) with issue reference.
Description check ✅ Passed PR description is largely complete with clear summary, component details, and test plan, but missing Storybook status and before/after screenshots as per template.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/LAI-5067-tag-icons-colorpicker

Comment @coderabbitai help to get the list of available commands and usage tips.

@mateusveloso Mateus Veloso (mateusveloso) merged commit a1065d8 into main Mar 30, 2026
2 of 3 checks passed
@fernandorjesus Fernando Jesus (fernandorjesus) deleted the feat/LAI-5067-tag-icons-colorpicker branch April 15, 2026 16:20
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