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)
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | multistep-active-li-class-1172874-6.patch | 871 bytes | fmoutawe |
| #6 | multistep-active-li-class-1172874-6.patch | 1.01 KB | fmoutawe |
| fix-classes.patch | 764 bytes | Peter Bex |
Comments
Comment #2
geek-merlinComment #3
Yuri commentedThe patch works nicely, thanks!
Comment #4
capellicPatch applies cleanly and works great. Let's get this into Dev! Thanks.
Comment #5
WorldFallz commentedanother confirmation. patch fixes the problem.
Comment #6
fmoutawe commentedWrong patch, see next patch.
Comment #7
fmoutawe commentedPatch 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.
Comment #8
bkno commentedOriginal patch works. Please get this into dev.
Thanks.
Comment #9
jollysolutions