[PP-1] Integrate the new Navigation Icon API with NavigationLinkBlock

Created on 17 February 2025, about 1 month ago

Problem/Motivation

Once 📌 Navigation leverage icon core API Needs work provides a new icon set for Navigation, the approach used by NavigationLinkBlock to define icon might need to be reviewed to make use of the new Icon API instead of the current magic class pattern.

Steps to reproduce

Proposed resolution

TBD

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

📌 Task
Status

Postponed

Version

11.0 🔥

Component

navigation.module

Created by

🇪🇸Spain plopesc Valladolid

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @plopesc
  • 🇨🇦Canada m4olivei Grimsby, ON

    This issue is still valid. We continue relying on the magic class pattern approach here. In my mind, ideally, we would allow the user to configure a icon_pack_id and icon_id pair for the block plugin that would get used in rendering it. That's an awkward UX experience, given core doesn't have any icon widgets, but, it could be left to contrib to make it better, and/or if/when core adopts a widget for icons.

Production build 0.71.5 2024