Make it happy :)

Comments

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Version: 6.x-1.x-dev » 7.x-2.x-dev
jensimmons’s picture

jensimmons commented
Status: Active » Needs review

Ok, we did a lot of work on this. It's all been pushed directly to the 7.x branch. People could review where we are at, and comment on any more changes they think we need.

alanburke’s picture

alanburke commented
Status: Needs review » Reviewed & tested by the community

Looks good.
I'm in two minds about removing all the comments that show all the variables available.

[I'd prefer to keep the calls to theme_menu away from the template, but that's a separate core issue]

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Status: Reviewed & tested by the community » Fixed

This is done.

Status: Fixed » Closed (fixed)

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