Navigation block with active class on ul

This should be targeted towards the 2.x branch.

I've tested the code and it works as expected, so for me it is RTBC. I created a new MR against 2.x with same code and tested it.

Please do not set RTBC by own patches without review of others.

Again: Thanks for the report and the work in here. Very much appreciated. But some nitpics:

-          $items[] = ['#markup' => Link::fromTextAndUrl($label, $url)->toString()];
+          if ($domain->isActive()) {
+            $url->mergeOptions(['attributes' => ['class' => 'active']]);
+          }
+          $items[] = [
+            '#type' => 'link',
+            '#url' => $url,
+            '#title' => $label,
+          ];

These changes do not only add active class in case but also change/replace the ['#markup' => Link::fromTextAndUrl($label, $url)->toString()]; without documentation in the issue comments. This should be documented and explained even if obvious for the patch provider, so that follow up issues can better be tracked down and find its root causes.

I only added the code that was already provided. but fair enough I follow that I can not set own patches/MR as RTBC.

What do you need from us to continue in this issue? Some sort of CR so everybody knows what happend and that can be linked in this issue and in the release notes? Or were you thinking about something else?

I rebased MR #75 from the current master branch, simplified fix and rewrote the case for testing.

Looks good but needs to be rebased.

Fixing.

agentrickard → changed the visibility of the branch 3226519-navigation-block-with-active-class to hidden.

agentrickard → changed the visibility of the branch 3226519-navigation-block-with-active-class to hidden.

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