Skip to content

Refactor/color converter structure#43

Merged
vuon9 merged 5 commits intomainfrom
refactor/color-converter-structure
Mar 1, 2026
Merged

Refactor/color converter structure#43
vuon9 merged 5 commits intomainfrom
refactor/color-converter-structure

Conversation

@vuon9
Copy link
Owner

@vuon9 vuon9 commented Feb 28, 2026

No description provided.

- Extract color utilities to colorUtils.js
- Extract reducer to colorReducer.js with action creators
- Create components directory with:
  - ColorInputRow: Individual color input with copy button
  - ColorInputs: Container for all color format inputs
  - ColorHistory: Color history panel with clear functionality
  - CodeSnippetsPanel: Language tabs with code snippets
- Update main index.jsx to use new structure
- Follow same pattern as JwtDebugger for consistency
- Remove old monolithic ColorConverter.jsx
Standardize indentation to 2 spaces, use single quotes consistently,
add trailing commas, and reformat JSX attributes. Update AGENTS.md
structure for clarity.
@vuon9 vuon9 force-pushed the refactor/color-converter-structure branch from cb1c723 to f631d3a Compare February 28, 2026 15:53
@vuon9 vuon9 force-pushed the refactor/color-converter-structure branch from 40db505 to c51686c Compare March 1, 2026 01:59
@github-actions
Copy link

github-actions bot commented Mar 1, 2026

Code Coverage

Package Line Rate Health
devtoolbox/internal/barcode 90%
devtoolbox/internal/codeformatter 56%
devtoolbox/internal/converter 60%
devtoolbox/internal/datagenerator 91%
devtoolbox/internal/datetimeconverter 73%
devtoolbox/internal/jwt 42%
Summary 65% (3083 / 4770)

Minimum allowed line rate is 60%

@vuon9 vuon9 marked this pull request as ready for review March 1, 2026 02:54
@vuon9 vuon9 merged commit 2ace856 into main Mar 1, 2026
2 checks passed
@vuon9 vuon9 deleted the refactor/color-converter-structure branch March 1, 2026 02:54
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.

1 participant