FXVPN-321

[lesleyjanenorton]

Fixes:

• FXVPN-321 - "Get help broken for all error panels"
• FXVPN-322 - "Subscribe now button does nothing"

Also:

• Changes the CTA on the last onboarding screen from 'Next' to 'Done'
• Hides 'skip' on the last onboarding screen

[mcleinman]

To make sure I understand, the initial telemetry request has moved from an onboarding screen to new tab? (It's not what is in the mocks, so I want to confirm this wasn't some sort of regression.)

[lesleyjanenorton]

To make sure I understand, the initial telemetry request has moved from an onboarding screen to new tab? (It's not what is in the mocks, so I want to confirm this wasn't some sort of regression.)

@mcleinman Exactly. This happened on Wednesday or Thursday of last week. New spec is here: https://www.figma.com/design/FixeQmo9NqzLa1sGtFWQ9s/Mozilla-VPN-Extension-Data-Collection-consent?node-id=8322-3458

[mcleinman]

Thanks!

If we didn't want to lock that hidden onboarding button to onboarding screen 3 (and make it dynamic, so it works as expected if we add a 4th screen in the future), we could do something like this. Definitely not necessary - especially because creating an unseen space is not the best practice, certainly.

[lesleyjanenorton]

I thought exactly the same thing but after ~20 minutes of attempting to add a class to that secondary action on the last onboarding panel I took the easy css way out.