Feature: IconPicker TypeScript Conversion

[Antonio-Laguna]

Description of the Change

This migrates the IconPicker to TypeScript and also removes defaultProps from it.

Relates to #295 and #314

How to test the Change

Changelog Entry

Changed - Moved IconPicker to TypeScript

Checklist:

• I agree to follow this project's Code of Conduct.
• I have updated the documentation accordingly.
• I have added tests to cover my change.
• All new and existing tests pass.

[github-actions]

Size Change: -48 B (-0.07%)

Total Size: 65 kB

Filename	Size	Change
dist/index.js	65 kB	-48 B (-0.07%)

_{compressed-size-action}

[cypress]

Passing run #846 ↗︎

0

6

0

0

0

Details:

Solidifying types
Project: 10up Block Components	Commit: 93138c769c
Status: Passed	Duration: 01:25 💡
Started: May 20, 2024 3:22 PM	Ended: May 20, 2024 3:24 PM

Review all test suite changes for PR #321 ↗︎

[Antonio-Laguna]

@fabiankaegy even though tests are passing I see a lot of things on the Cypress run. While there are things that look unrelated, the related ones are odd. I based the code on what was there but those type checking might be coming from WP code or something (see dashicons mention)

[fabiankaegy]

@Antonio-Laguna yeah currently the npm run build command spits out thousands of errors because the types are still very rough and incomplete.

I will review this in a bit :) Thanks for your help <3

[fabiankaegy]

@Antonio-Laguna testing this locally I am seeing the following error when trying to add className to the <InlineIconPicker

block-components/example/src/blocks/icon-picker-example/edit.js

Line 43 in da1cb5f

<InlineIconPicker value={icon} onChange={handleIconSelection} className="icon-preview"/>

[Antonio-Laguna]

@fabiankaegy solved :)

[fabiankaegy]

This looks great :) Thanks!