Skip to content

Conversation

@ThomasLandauer
Copy link
Contributor

Q A
Bug fix? no
New feature? no
Docs? yes
Issues
License MIT

Page: https://symfony.com/bundles/ux-icons/current/index.html#configuration

I moved "Debugging" to the bottom (didn't change anything).

And then the first (short) code block isn't needed anymore, since the second shows it all.

Page: https://symfony.com/bundles/ux-icons/current/index.html#configuration

I moved "Debugging" to the bottom (didn't change anything).

And then the first (short) code block isn't needed anymore, since the second shows it all.
@carsonbot carsonbot added Icons Status: Needs Review Needs to be reviewed labels May 29, 2025
@carsonbot carsonbot added Status: Reviewed Has been reviewed by a maintainer and removed Status: Needs Review Needs to be reviewed labels May 29, 2025
@smnandre
Copy link
Member

You just removed the configuration from the .... "configuration".

See my comment/interrogation on the other PR.

👎 for me here, and i'm sorry and would gladly discuss the whole situation.

@smnandre smnandre closed this May 29, 2025
@ThomasLandauer
Copy link
Contributor Author

Sorry, I don't know what you mean! Here's the configuration: https://github.com/ThomasLandauer/ux/blob/ad94afd450a2a895eaae6efbd05888f9a8125364/src/Icons/doc/index.rst#L553

@smnandre
Copy link
Member

image

Before:

Configuration

  • how it's done
  • where it's done
  • how to debug
  • full reference

After

Configuration

  • it exists
  • here is a whole yaml, not matching what users have locally
  • ho and by the way, if you are still reading, here are the command to see your configuration

@ThomasLandauer ThomasLandauer deleted the patch-7 branch May 30, 2025 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Icons Status: Reviewed Has been reviewed by a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants