Skip to content

Custome RichText: Pick an icon from Icons ref - #3330

Merged
adriendupuis merged 6 commits into
5.0from
rte-icons
Jul 30, 2026
Merged

Custome RichText: Pick an icon from Icons ref#3330
adriendupuis merged 6 commits into
5.0from
rte-icons

Conversation

@adriendupuis

Copy link
Copy Markdown
Contributor
Question Answer
JIRA Ticket
Versions
Edition

Link to Icons reference to help picking an out of the box icon.

Discovered while working with Copilot to create a custom tag, without a link, it improvised an icon ID "bookmark" which doesn't exist. After telling him "you should find a Icons reference table somewhere in the doc", it was able to propose existing icons.

It doesn't thought by itself that a link is the missing solution but it agreed that it could help.

User

What could have helped you to not use non-existing "bookmark" icon in the first place?


82m 21s

Copilot

Looking at the icon reference table in the documentation — which I actually found and used to fix the issue — I should have consulted it before choosing the icon name, not after. The table at vendor/ibexa/documentation-developer/developer/templating/twig_function_reference/icon_twig_functions/index.md lists all valid icon IDs, and a quick grep for bookmark there would have immediately shown that bookmark alone doesn't exist, while bookmark-filled, bookmark-outline, and bookmarks do.

Additionally, grepping directly against the SVG sprite (all-icons.svg) for id="bookmark" would have caught the mismatch before writing the config.


83m 43s

User

Would a link to available icons in the doc you used to create the custom tag would have help? If yes, which doc page?


83m 46s

Copilot

Yes, a direct link would have helped. The "Extend Online Editor" page (vendor/ibexa/documentation-developer/developer/content_management/rich_text/extend_online_editor/index.md) mentions an icon key in the custom tag config examples, but never links to the available icon names.

Adding a reference there to the icon reference table at:

developer/templating/twig_function_reference/icon_twig_functions/index.md#icons-reference

— the same link already present in the custom icons page — would have prompted me to look up the exact ID before using a guessed name.

Checklist

  • Text renders correctly
  • Text has been checked with vale
  • Description metadata is up to date
  • Redirects cover removed/moved pages
  • Code samples are working
  • PHP code samples have been fixed with PHP CS fixer
  • Added link to this PR in relevant JIRA ticket or code PR

@github-actions

Copy link
Copy Markdown

Preview of modified files

Preview of modified Markdown:

@github-actions

github-actions Bot commented Jul 29, 2026

Copy link
Copy Markdown

Summary

Status Count
🔍 Total 736641
🔗 Unique 14760
✅ Successful 6190
⏳ Timeouts 0
🔀 Redirected 3
👻 Excluded 730450
❓ Unknown 0
🚫 Errors 1
⛔ Unsupported 0

Errors per input

Errors in site/resources/new_in_doc/index.html

  • [ERROR] file:///home/runner/work/documentation-developer/documentation-developer/repositories/userdoc-4.6/site/content_management/block_reference/index.html#bestsellers-block (at 19441:14) | Cannot find fragment

Redirects per input

Redirects in site/ai/mcp/mcp_guide/index.html

Redirects in site/ai/mcp/mcp_usage/index.html

Full Github Actions output

@adriendupuis
adriendupuis marked this pull request as ready for review July 30, 2026 07:32
Comment thread docs/content_management/rich_text/extend_online_editor.md Outdated
adriendupuis and others added 2 commits July 30, 2026 10:09
Co-authored-by: Marek Nocoń <mnocon@users.noreply.github.com>
@adriendupuis
adriendupuis requested a review from mnocon July 30, 2026 08:11
Comment thread docs/content_management/rich_text/extend_online_editor.md Outdated
Comment thread docs/content_management/rich_text/extend_online_editor.md Outdated
Co-authored-by: Adrien Dupuis <61695653+adriendupuis@users.noreply.github.com>
@adriendupuis
adriendupuis merged commit 37c165c into 5.0 Jul 30, 2026
11 of 12 checks passed
@adriendupuis
adriendupuis deleted the rte-icons branch July 30, 2026 09:06
adriendupuis added a commit that referenced this pull request Jul 30, 2026
---------

Co-authored-by: Marek Nocoń <mnocon@users.noreply.github.com>
adriendupuis added a commit that referenced this pull request Jul 30, 2026
---------

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants