For homepage variations like CWV, Compare, Tracerout, Lighthouse, we're not setting nav state and content include is broken #2334
Assignees
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Seems like two issues here, possibly a regression from when we started varying the homepage content for pro users.
First, the
Start Teststate in the nav should be set for all these pages, and it seems they never were. This isn't a regression but we can fix it like this:https://github.com/WPO-Foundation/webpagetest/blob/7e85dc41099d1fa5fb4451f912954dae73bb4529/www/webvitals.php#L45
This line should be
$tab = "Start Test";Second, when you're logged in as PRO, the content header is not there on these pages. I think all that is needed is these pages need a body class of
<body class="home feature-pro">The text was updated successfully, but these errors were encountered: