🇫🇷France @mogtofu33

Seems like just removing the specific light classes is enough for the dark theme support!

Small update, it can be achieved now with ui_icons_media using the core CKeditor media plugin.

• With ui_icons_media enable, create an media type icon
• Create the limited number of icons you need
• Media Icon fields > Manage display > Media library, replace thumbnail by icon with settings pack size to 50
• Text formats and editors > enable Media, limit to type Icon

Contrib modules around media embed can help customize.

Can you be more precise on how to reproduce, this is related to link with icon as a field in a media?

mogtofu33 → created an issue.

mogtofu33 → changed the visibility of the branch 3489673-regression-with-uiiconsmenu to hidden.

Because this is now in core, we can close.

Pushed but I cannot create the MR.

Pushed but I cannot create the MR.

mogtofu33 → created an issue.

(Try to) Fix code language YAML overridden by language PHP.

Remove SVG sprite url reference as it's not currently available.

Add properties and parameters for icon element and twig function.

Faster on both sides, for discovery and a better in front at least after first download...
SVG Sprite could be use with CDN but not public one because of CORS, so it's an edge case optimization.

As it's just a suggestion I understand we could keep to SVG. My mind is not fully set on that.

mogtofu33 → created an issue.

mogtofu33 → created an issue.

mogtofu33 → created an issue.

Fixed on 1.1.x, backported on 1.0.x.

mogtofu33 → created an issue.

Had an issue with a double icon_display, fixed in IconFieldHelpers::validateSettings(), fixed on 1.1.x, backported 1.0.x.

But didn't had double pack settings.

Please try to export config, manually edit to remove what's not under icon_settings and import it back.

Here is my test on clean Drupal 11.1 + UI icons 1.1.x-dev.

• Enable UI Icons media, Icons bootstrap example with this config and create Icon media
• Set media icon field display settings for the icon packs
• Add field Media icons to content
• Set content display with layout builder

With that, config core.entity_view_display.media.icon.default is valid as below.

content:
  field_linkit_attributes:
    type: icon_linkit_formatter
    label: hidden
    settings:
      trim_length: 80
      url_only: false
      url_plain: false
      rel: '0'
      target: _blank
      linkit_profile: default
      icon_settings:
        bootstrap:
          color: '#6200ee'
          size: 32
          alt: ''
        bootstrap_svg:
          color: '#f66151'
          size: 32
        bootstrap_path:
          color: '#000000'
          size: 32
          alt: ''
        bootstrap_name:
          color: '#000000'
          size: 32
      icon_display: after
    third_party_settings: {  }
    weight: 0
    region: content

Found the doc! Fixed on 1.0.x and 1.1.x

Done for most, only Protocol and Material missing, issues ongoing.

Added to module page and Drupal Icon API doc → .

mogtofu33 → created an issue.

As for now this field is not planned for UI icons, if anyone want to create a dedicated modules, would review.

There is not token integration yet but you can achieve this with Ui Icons field.

• Add Field type icon to the vocabulary
• Add icon field to the taxonomy display
• On the content field > Manage display > Taxonomy field set to "Rendered entity", pick the previously set taxonomy display

Thanks for the report, fixed in 1.0.x and 1.1.x documentation, added to project page.

Thanks for the report.

Fixed on 1.1.x and backported to 1.0.x.

Thanks for the report, changed for an 'access content' permission.

Fixed on 1.1.x and backported to 1.0.x.

Fixed on 1.1.x, backported to 1.0.x.

There is already a lazy loading for Icon preview (Form element) and library.

For the front part if you need compatibility with Blazy you only need to have a template for your icons including the proper markup, see Icon API documentation → .

So I guess it's already compatible, marked as fixed.

Can only reproduce when a pack use extractor: font and do not have dependency to ui_icons_font.
Fixed in the examples for Bootstrap and Material.

Will mark this issue fixed, feel free to open with step to reproduce if the problem still occur.

Fix YAML examples, a specific section "YAML icon pack declaration documentation" explain in details most keys.

Thanks for the proposal, done and created 1.1.x, 1.1.0-beta1 to come soon.

Thanks for the report, fixed on 1.0.x-dev.

mogtofu33 → made their first commit to this issue’s fork.

mogtofu33 → made their first commit to this issue’s fork.

mogtofu33 → made their first commit to this issue’s fork.

Hello all, here is an highlight proposition:

A new API has been added to Drupal, empowering you to seamlessly integrate custom or third-party icon packs!
This powerful and flexible API allows modules and themes to define icon packs using the YAML plugin discovery system (EXTENSION_NAME.icons.yml).
By extracting the icon list from each pack, the API exposes them through the versatile icon render element and the convenient icon() Twig function.

grimreaper → credited mogtofu33 → .

smustgrave → credited mogtofu33 → .

Seems fixed with beta3, to confirm.

Looks good, thanks!

Bug is back, with ui_icons_field, if display field icon is set to Link (not Link icon) the display has the Icon because of ui_icons_menu_link_alter .

Working but slow, find why.

Hi @steveoriol, with 1.0.x-dev you need to enable module ui_icons_backport, it will be in release note on beta3, it's because the api is now in Drupal 11.1.x.

Good for me.

mogtofu33 → created an issue.

mogtofu33 → created an issue.

mogtofu33 → created an issue.

@catch, thanks for the review, cache collector fixes and merge.

@grimreaper, good idea, added link to ui_icons and tests. I think relate to an external repository for examples should be avoided.

Hi @johnwebdev, change records updated with more simple examples, let me know if it's more clear.

mogtofu33 → created an issue.

mogtofu33 → created an issue.

Ther is a dependency on ui_patterns_library for the stories and preview. but I think it can be a soft dependency and optional.

mogtofu33 → created an issue.

mogtofu33 → created an issue.

mogtofu33 → created an issue.

Adapted a bit and update the test. Thanks.

mogtofu33 → made their first commit to this issue’s fork.