Skip to content

Conversation

@adamwojs
Copy link
Member

@adamwojs adamwojs commented Feb 2, 2025

Preview:
https://ez-systems-developer-documentation--2602.com.readthedocs.build/en/2602/templating/twig_function_reference/other_twig_filters/#ibexa_taxonomy_entries_for_content

Target: master, 4.6

Command used to generate icons preview:

find *.svg -type f -exec convert -background none -resize 32x32 {} {}.png \;

@adamwojs adamwojs force-pushed the icons_reference branch 2 times, most recently from eea873a to e87887f Compare February 2, 2025 18:23
@adamwojs adamwojs marked this pull request as ready for review February 14, 2025 15:13
Copy link
Contributor

@mnocon mnocon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@@ -43,6 +43,338 @@ The list of available icon sizes:
|`48px`|`--large`|
|`64px`|`--extra-large`|

#### Icons reference
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@adamwojs While we are at it, what does the --light switch do and what are other options (am I right in guessing it is dark lines on white background, and the other option is --dark with white lines on black)?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess the other options are here:
https://github.com/ibexa/admin-ui/blob/main/src/bundle/Resources/public/scss/_icons.scss#L45-L64

@dew326 do you see any other options worth documenting here, except the sizes?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, only sizes and colors.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dew326 @dabrt section about colors added in b36406c
(already consulted internally with Darek)

@github-actions
Copy link

Preview of modified Markdown:

@dabrt dabrt self-requested a review February 20, 2025 10:55
@mnocon mnocon merged commit 498ad69 into master Feb 20, 2025
4 of 6 checks passed
@mnocon mnocon deleted the icons_reference branch February 20, 2025 11:02
mnocon added a commit that referenced this pull request Feb 20, 2025
* Added icons reference

* Added section about colors

* After review

---------

Co-authored-by: Marek Nocoń <mnocon@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants