- Issue created by @baluv3
- 🇫🇮Finland lauriii Finland
There's some work in progress in 📌 Generate menu items using plugins RTBC .
- 🇪🇸Spain ckrina Barcelona
Just added 🌱 Define logo origin on the Toolbar Active since we'll likely have to define this soon.
- 🇺🇸United States mherchel Gainesville, FL, US
Just merged 📌 Generate menu items using plugins RTBC .
Had discussions that fit into this issue with @deviantintegral and @tedbow at https://drupal.slack.com/archives/C7AB68LJV/p1695052420501349.
Both of them are confused on where the data is coming from.
From @tedbow
I am little confused about the purpose of this issue vs using the menu system. It sort of uses the menu system, as in NavigationAdmin uses a menu but NavigationUser does not. Eventually will these all use menus? If so couldn’t you have just as much control over the markup by just having specific templates for these menu rather than just introducing the new NavigationSection concept
does 'base hook' => 'menu', mean that all the theme process stuff will still apply to these navigation sections also? (edited)from @deviantintegral
I was also a little confused about this, but assumed I'd just missed prior context. The big thing I see is that I could see modules adding items to navigation sections that aren't menus, but some other UI widget. Like I think environment indicator is a good example of that? But I also think until we start wanting to create releases there isn't much harm in merging the MR, given that it's less hardcoded than the current code.
- 🇪🇸Spain ckrina Barcelona
Updating the plan with an initial list of issues
- 🇪🇸Spain ckrina Barcelona
Update the roadmap and the work done so far.
- Status changed to Fixed
9 months ago 1:12pm 6 April 2024 - 🇺🇸United States tedbow Ithaca, NY, USA
ckrina → credited tedbow → .
- 🇺🇸United States tim.plunkett Philadelphia
ckrina → credited tim.plunkett → .
- 🇪🇸Spain ckrina Barcelona
Closing the plan and leaving 📌 Add contextual link for the whole Navigation bar to link to the Navigation Admin UI Needs work as its stand-alone issue, because we're already using menus to generate the links. Since 📌 Add contextual link for the whole Navigation bar to link to the Navigation Admin UI Needs work will need its own discussions more focused on UX, I'm detaching it from the plan itself.
Automatically closed - issue fixed for 2 weeks with no activity.