This is due to the fact that the l() function adds 'active' to each URL that matches $_GET['q'], and ignores the differentiation of the 'query' arg.

The attached patch works around this active check by forcing an "external" fully qualified URL into l(), so it cannot determine that it's the active page. Multistep already contains the code to properly add an 'active' class to the really active step (as you can see by the fact the link gets two active classes assigned)

Comments

geek-merlin’s picture

Issue tags: -links, -styling, -active, -classes +multistep-d7review
Yuri’s picture

The patch works nicely, thanks!

capellic’s picture

Status: Needs review » Reviewed & tested by the community

Patch applies cleanly and works great. Let's get this into Dev! Thanks.

WorldFallz’s picture

another confirmation. patch fixes the problem.

fmoutawe’s picture

StatusFileSize
new1.01 KB

Wrong patch, see next patch.

fmoutawe’s picture

Status: Reviewed & tested by the community » Needs review
StatusFileSize
new871 bytes

Patch fixes the problem, but add only active class on <a> element.
I make a patch to add active class on <li> element too.

The previous patch has a dpm, sorry.
This is the good one.

bkno’s picture

Issue summary: View changes

Original patch works. Please get this into dev.

Thanks.

jollysolutions’s picture

Status: Needs review » Reviewed & tested by the community