Skip to content
This repository has been archived by the owner on Aug 11, 2021. It is now read-only.

Shared signup #349

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Shared signup #349

wants to merge 5 commits into from

Conversation

vincaslt
Copy link
Contributor

@vincaslt vincaslt commented Oct 15, 2019

Related:
proton-mail: ProtonMail/proton-mail#14
proton-vpn-settings: ProtonVPN/proton-vpn-settings#304

Works in mail/vpn, but there is some missing stuff until it's done:

  • Mail plans features/icons/descriptions texts
  • Dynamic login modal texts (when registering with protonmail/vpn account)
  • Dynamic links (header, routes)
  • Dynamic header (show mail logo)
  • Plans table improvements (expanded, no plan selection)

Variations are based on CLIENT_TYPE from useConfig.

--
Mat said to hold on with it, because mail signup will have some different stuff, and UX is still figuring it out.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant