* [review comment] move header button active setting to router and out of views