# Soil Nav Walker issue

**URL:** https://discourse.roots.io/t/soil-nav-walker-issue/13496
**Category:** sage
**Created:** 2018-08-30T13:06:24Z
**Posts:** 2

## Post 1 by @paulrose — 2018-08-30T13:06:24Z

Hi Roots team

I’ve found an issue with the nav-walker in Soil. When I create a menu, with a sub menu, the parent menu-item gets an ‘active’ class added.

If I change line 43 to read like this:

```
if (($this->archive && $child->url) && ($child->current_item_parent || Utils\url_compare($this->archive, $child->url))) {
```

then it doesnt add the **active** class to the parent anymore, and does so correctly when I browse to that parent page, and correctly so to a sub page of that parent.

Make sense?

Not sure if this is because I have the menu structure like this (to build a mega nav):

Parent Item  
– Parent Item  
---- Sub Item  
---- Sub Item

---

## Post 2 by @ben — 2018-08-30T20:00:32Z

Hey Paul!

I believe it’s meant to be like this since all any classes with `current` in the name get turned into `active`. On a default WP install you’d see `current-page-ancestor` and `current-page-parent` (or similar) on your nav items