-
-
Notifications
You must be signed in to change notification settings - Fork 372
[Icons] Document icon_sets
configuration
#2938
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
# Icon sets configuration, array of prefix => icon set configuration | ||
icon_sets: | ||
# Icon set for "flags", can be used like this: ux_icon('flags:fr') | ||
flags: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't add any "added in 2.20", this version is pretty old right now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
flags: | ||
It can either be mapped to a local directory... | ||
path: '%kernel.project_dir%/assets/images/flags' | ||
... or to an existing icon set identifier |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
... or to an existing icon set identifier | |
# ... or to an existing icon set identifier |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in #2939, thanks!
icon_sets: | ||
# Icon set for "flags", can be used like this: ux_icon('flags:fr') | ||
flags: | ||
It can either be mapped to a local directory... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It can either be mapped to a local directory... | |
# It can either be mapped to a local directory... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤦🏻
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in #2939, thanks!
#2938 (Kocal) This PR was merged into the 2.x branch. Discussion ---------- [Icons] Fix `icon_sets` configuration documentation, follow #2938 | Q | A | ------------- | --- | Bug fix? | no | New feature? | no <!-- please update src/**/CHANGELOG.md files --> | Docs? | yes <!-- required for new features --> | Issues | Fix #... <!-- prefix each issue number with "Fix #", no need to create an issue if none exist, explain below instead --> | License | MIT <!-- Replace this notice by a description of your feature/bugfix. This will help reviewers and should be a good start for the documentation. Additionally (see https://symfony.com/releases): - Always add tests and ensure they pass. - For new features, provide some code snippets to help understand usage. - Features and deprecations must be submitted against branch main. - Update/add documentation as required (we can help!) - Changelog entry should follow https://symfony.com/doc/current/contributing/code/conventions.html#writing-a-changelog-entry - Never break backward compatibility (see https://symfony.com/bc). --> Following #2938 Commits ------- f5884b3 [Icons] Fix `icon_sets` configuration documentation, follow #2938
Related to #2156