Skip to content

fix(react-icons-file-type): map clpx extension to clipchamp icon - #1203

Merged
C. Perez (bigbadcapers) merged 5 commits into
microsoft:mainfrom
bigbadcapers:caperez/clpx-clipchamp-icon
Aug 24, 2026
Merged

fix(react-icons-file-type): map clpx extension to clipchamp icon#1203
C. Perez (bigbadcapers) merged 5 commits into
microsoft:mainfrom
bigbadcapers:caperez/clpx-clipchamp-icon

Conversation

@bigbadcapers

Copy link
Copy Markdown
Collaborator

Previous behavior

The clipchamp entry only mapped the clipchamp extension, so .clpx files resolved to the generic file icon.

New behavior

The clpx extension now maps to the existing Clipchamp file type icon. No new icon asset or FileIconType entry is required.

This mirrors microsoft/fluentui#36507 for @fluentui/react-icons-file-type.

Validation

  • Full react-icons-file-type Vitest suite: 5 files, 42 tests passed
  • ESLint: passed for packages/react-icons-file-type/src
  • Prettier: passed for all changed files
  • TypeScript build: ESM and CommonJS compilations passed

@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown

📋 PR Validation Summary

Check the Build react library job summary for detailed reports:

  • 📦 Bundle Size — size comparison against the base branch
  • 📖 Docsite Preview — build artifact with local preview instructions

To view: click the link above → select the Build react library job → open the Summary tab.

@bigbadcapers
C. Perez (bigbadcapers) marked this pull request as ready for review August 5, 2026 18:33
@bigbadcapers
C. Perez (bigbadcapers) requested review from a team as code owners August 5, 2026 18:33

@Hotell Martin Hochel (Hotell) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@bigbadcapers
C. Perez (bigbadcapers) merged commit d7e73bf into microsoft:main Aug 24, 2026
7 checks passed
@bigbadcapers
C. Perez (bigbadcapers) deleted the caperez/clpx-clipchamp-icon branch August 25, 2026 21:59
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.

3 participants