Conditional navigation items

[paulrobertlloyd]

Makes it possible to add conditions to navigation items:

{{ xGovukSecondaryNavigation({
  classes: "nhsuk-u-margin-bottom-6",
  items: [
    {
      text: "Apples",
      href: "/apples/"
    } if role == "juicer",
    {
      text: "Bananas",
      href: "/bananas/"
    },
    {
      text: "Oranges",
      href: "/oranges/"
    } if role == "juicer"
  ]
}) }}

Without this change, an empty object is returned, which in turn creates an empty HTML list item.

The Primary navigation component previously checked for the presence of item.href, but have made this consistent so that only item need be present.

Also updated the indentation to be a bit more consistent across the navigation components, see edbb9f6 for the actual changes.

[frankieroberto]

Ah, I always forget about that inline condition trick! Nice. Might be worth documenting somewhere too?

Hard to review as you changed all the indentation at the same time 😛, but looks good. Thanks!

[paulrobertlloyd]

Hard to review as you changed all the indentation at the same time 😛

Cough:

Also updated the indentation to be a bit more consistent across the navigation components, see edbb9f6 for the actual changes.

[frankieroberto]

@paulrobertlloyd heh, I somehow missed that last para and went straight to the Files changed tab... 🤦