Automattic / Automattic/jetpack
Blocks: use JetpackLogo from Jetpack components
- Dominant language
- PHP
- Stars
- 1.8k
- Forks
- 898
- Avg merge
- 1d 18h
- Merged PRs (30d)
- 774
Description
The block bundle in the Jetpack plugin has a `JetpackLogo` that is used to render the Jetpack logo in multiple places:
https://github.com/Automattic/jetpack/blob/94bc84fd2dd774b8237c995d67decd5a323d8a3d/projects/plugins/jetpack/extensions/shared/icons.js#L187
This works fine, but we now have a `JetpackLogo` in use in projects leveraging RNA:
https://github.com/Automattic/jetpack/blob/3238c5d6df6727ce64fc18763a0025f10920a317/projects/js-packages/components/components/jetpack-logo/index.jsx#L13
It would be nice if the block bundle could use that instead.
Contributor guide
Research direction
Start with projects/plugins/jetpack/extensions/shared/icons.js at the linked JetpackLogo definition, then compare it with projects/js-packages/components/components/jetpack-logo/index.jsx. Update the block bundle to use the shared Jetpack component, and verify that the existing places rendering the logo still use the intended component.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, react
- Domain
- frontend
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 48/100