Restrict configuration of the Navigation bar footer items

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

Raised MR https://git.drupalcode.org/issue/navigation-3411099/-/merge_requests/1 and created the merge request against the https://git.drupalcode.org/issue/navigation-3411099/-/tree/3411099-creat....

Kindly review.

Thanks @Prashant.c. Couple things.

1. Lets hide the region column on the Navigation section UI, since there is only once choice. Probably change the form element to a '#type' => 'value'.

2. Other note I left as a MR comment wrt the entity form.

@m4olivei

As soon as Iam changing the form element from "select" to "value" or something else the tabledrag stops working, not able to drop items and JS error The navigation section cannot be placed in this region.

Any ideas?

Nothing immediately comes to mind, I would have to play with it. May get time for that next week. No rush, would like to see 📌 Create an administration UI for managing Navigation Sections Needs work land first anyway.

I'm thinking whether it's okay to use CSS here to hide the label and select list for regions.

I'm thinking whether it's okay to use CSS here to hide the label and select list for regions.

I don't think that's a great idea, you still have the form value you need to deal with on the back end.

I've pushed a couple of commits. First to remove the region column from the Navigation Section admin UI. That comes with necessary adjustments to the #tabledrag config and Javascript handling that we no longer need. Second, to change the #type select to #type value on the NavigationSectionForm.

I believe that puts us in a good place here. @Prashant.c, please review.

As we discussed in Slack, we will hold off on merge here until the main issue ( 📌 Create an administration UI for managing Navigation Sections Needs work ) is resolved and merged to 1.x.

I needed to rebase the branch against 1.x of the navigation module. I also needed to open a new MR (linked in the ticket description) b/c Gitlab wouldn't let me det the target fork for the MR that @Prashant.c opened. I've closed that one.

@Prashant.c if you could review, that would be wonderful!

m4olivei → changed the visibility of the branch 3412119-restrict-configuration-of to hidden.

m4olivei → changed the visibility of the branch 3412119-restrict-configuration-of to active.

ckrina → closed merge request !159

ckrina → reopened merge request !159

Merged! Thanks for the work!

Automatically closed - issue fixed for 2 weeks with no activity.