active class must be on li not achor