Skip to content

feat: add LoadFileButton component and integrate it into DiffChecker,…#72

Merged
drakehanguyen merged 1 commit intomainfrom
feature/load-file-and-diff-alignment
Mar 28, 2026
Merged

feat: add LoadFileButton component and integrate it into DiffChecker,…#72
drakehanguyen merged 1 commit intomainfrom
feature/load-file-and-diff-alignment

Conversation

@drakehanguyen
Copy link
Copy Markdown
Collaborator

… JsonFormatter, JsonPathFinder, and XmlFormatter tools

  • Introduced LoadFileButton for file uploads across multiple tools.
  • Updated DiffChecker, JsonFormatter, JsonPathFinder, and XmlFormatter to utilize LoadFileButton for loading files, enhancing user experience.
  • Improved header actions in JsonFormatter and XmlFormatter to include file loading functionality.

Description

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📝 Documentation update
  • ♻️ Code refactoring (no functional changes)
  • ⚡ Performance improvement
  • ✅ Test addition or update
  • 🎨 Style/formatting changes (no functional changes)
  • 🔧 Configuration changes
  • 🔒 Security fix

Related Issue

Closes #
Related to #

Changes Made

Testing

  • ✅ All existing tests pass
  • ✅ New tests added for new functionality
  • ✅ Manual testing completed
  • ✅ Tested on multiple browsers (if applicable)
  • ✅ Tested on mobile devices (if applicable)

Test Steps

Screenshots (if applicable)

Before After

Checklist

Code Quality

  • Code follows the project's style guidelines
  • Self-review of code completed
  • Comments added for complex code/logic
  • No console.log statements left in code
  • No commented-out code
  • Code is properly formatted (Prettier)

TypeScript

  • TypeScript types are properly defined
  • No any types used (unless necessary)
  • Type checking passes (pnpm type-check)

Linting & Formatting

  • ESLint passes (pnpm lint)
  • Code is properly formatted (pnpm format:check)

Testing

  • Unit tests added/updated
  • All tests pass (pnpm test)
  • Test coverage maintained or improved

Documentation

  • README.md updated (if applicable)
  • CHANGELOG.md updated (if applicable)
  • Code comments added/updated
  • JSDoc comments added for public APIs (if applicable)

Build & Deployment

  • Build succeeds (pnpm build)
  • No build warnings or errors
  • Production build tested locally

Accessibility

  • Accessibility considerations addressed
  • Keyboard navigation works (if applicable)
  • Screen reader tested (if applicable)

Browser Compatibility

  • Tested on Chrome
  • Tested on Firefox
  • Tested on Safari
  • Tested on Edge (if applicable)
  • Mobile responsive (if applicable)

Performance

  • No performance regressions
  • Bundle size impact considered
  • Lazy loading used where appropriate

Additional Notes

Breaking Changes

Migration Guide

… JsonFormatter, JsonPathFinder, and XmlFormatter tools

- Introduced LoadFileButton for file uploads across multiple tools.
- Updated DiffChecker, JsonFormatter, JsonPathFinder, and XmlFormatter to utilize LoadFileButton for loading files, enhancing user experience.
- Improved header actions in JsonFormatter and XmlFormatter to include file loading functionality.
@drakehanguyen drakehanguyen merged commit 38c42e2 into main Mar 28, 2026
7 checks passed
@drakehanguyen drakehanguyen deleted the feature/load-file-and-diff-alignment branch March 28, 2026 05:16
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