Added labelColor, logo, logoColor, new style - social

[aptyp4uk1337]

I've added the features everyone has been asking for:

• Added labelColor
• Added logo (image URL / data:image)
• Added logoColor
• Added social style

I've tested it, and everything should work, but I recommend double-checking everything and making any necessary adjustments if something isn't right.

[Copilot]

Pull request overview

This PR extends the badge rendering system to support additional customization options (labelColor + logo/logoColor) and introduces a new “social” badge style, along with a renderer refactor to share common SVG rendering behavior.

Changes:

• Add labelColor, logo, and logoColor parsing/storage on Badge and expose them to SVG rendering via shared parameter building.
• Add a new social SVG template + renderer, and update existing templates to support logoElement and refreshed layout/typography.
• Refactor renderers by introducing SvgBaseRenderer and enhancing LocalSvgRenderer (logo element generation, parameter changes, and SVG output normalization).

Reviewed changes

Copilot reviewed 11 out of 16 changed files in this pull request and generated 10 comments.

Show a summary per file

File	Description
src/UI/Command.php	Registers the new social renderer in the CLI command’s Poser.
src/Resources/templates/social.svg	Adds a new SVG template for the “social” style.
src/Resources/templates/plastic.svg	Updates plastic template to new layout + optional logoElement.
src/Resources/templates/for-the-badge.svg	Updates for-the-badge template to uppercase vars + optional logoElement.
src/Resources/templates/flat.svg	Updates flat template to new layout + optional logoElement.
src/Resources/templates/flat-square.svg	Updates flat-square template to new layout + optional logoElement.
src/Render/SvgSocialRender.php	Introduces renderer-specific geometry for “social” style (pill gap + text shift).
src/Render/SvgPlasticRender.php	Refactors to inherit common template-name behavior via SvgBaseRenderer.
src/Render/SvgForTheBadgeRenderer.php	Refactors sizing/centering logic and logo Y positioning for 28px height.
src/Render/SvgFlatSquareRender.php	Refactors to inherit common template-name behavior via SvgBaseRenderer.
src/Render/SvgFlatRender.php	Refactors to inherit common template-name behavior via SvgBaseRenderer.
src/Render/SvgBaseRenderer.php	Adds shared “template name = style” base implementation.
src/Render/LocalSvgRenderer.php	Adds logo support + new width/position parameter calculations + output normalization.
src/Poser.php	Makes getRenderFor() public.
src/Calculator/SvgTextSizeCalculator.php	Adds dependency checks and better error reporting.
src/Badge.php	Adds labelColor/logo/logoColor support and accepts 3-digit hex colors.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[garak]

Hello @aptyp4uk1337 thank you for your contribution.
Could you please address all the problems raised by the review?
Please also fix the CI failures.

Thank you

[aptyp4uk1337]

Hello @aptyp4uk1337 thank you for your contribution. Could you please address all the problems raised by the review? Please also fix the CI failures.

Thank you

I had to change tests/Fixtures because they weren't passing. I also used bin/php-cs-fixer to format the code.
I ran tests using bin/phpunit and bin/behat, and there were no errors. I also had to make some changes to .github/workflows/php.yml because Docker kept throwing errors during the warnings. The last test I ran passed without any issues.

[antonkomarev]

If we are changing fixtures, it means that we are breaking BC. And do we really need to change CI file?

This PR is pretty complex and huge. Maybe it will be better to split it for a separate feature branches, that solves only one thing at a time. I mean at least does not mix new social badge to this PR.

[aptyp4uk1337]

If we are changing fixtures, it means that we are breaking BC. And do we really need to change CI file?

This PR is pretty complex and huge. Maybe it will be better to split it for a separate feature branches, that solves only one thing at a time. I mean at least does not mix new social badge to this PR.

The fixtures doesn't seem like any test checks the social media icon at all—there shouldn't be any issues with it.

I had issues with the fixtures because the SVG structure didn't match the existing one.

There were issues with the code structure, which were fixed using the bin/php-cs-fixer command.

The other issues were that I couldn’t run GitHub Actions locally via Docker for the check; there were some problems with packages and paths, so I had to make changes to .github/workflows/php.yml as well, after which all tests passed without errors.

[JellyBellyDev]

Fantastic work! Thank you so much!
LGTM

[JellyBellyDev]

@aptyp4uk1337 only after generating the tag did I realize that we forgot to add documentation on how to use the new features!
Would you like to take care of it?
Thanks!