Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Theme Card: How to show that a theme is already purchased? #65704

Open
andres-blanco opened this issue Jul 18, 2022 · 6 comments
Open

Theme Card: How to show that a theme is already purchased? #65704

andres-blanco opened this issue Jul 18, 2022 · 6 comments

Comments

@andres-blanco
Copy link
Contributor

If a theme has already been purchased then we need a new visual state for the Theme Card to reflect this.

Related to #65567

@blackjackkent
Copy link
Contributor

@SaxonF Another styling element of theme subscription work that we could use some design input on when you have a chance. 🙂

@SaxonF
Copy link
Contributor

SaxonF commented Jul 19, 2022

Theme cards within onboarding
image

Theme cards within theme showcase
image

In future we'll have separate sections for exploring themes and managing purchased themes so this will be less of an issue.

Thoughts?

@andres-blanco
Copy link
Contributor Author

@SaxonF Do we need a "Already purchased" state on the theme preview screen?

This one:
image

@SaxonF
Copy link
Contributor

SaxonF commented Jul 20, 2022

@andres-blanco not for this first release. We can re-access after.

@mreishus
Copy link
Contributor

mreishus commented Aug 1, 2022

For the theme cards in onboarding, after #66135 is merged, we can split the !shouldUpgrade text, Included in your plan, to both Purchased on an [annual|monthly subscription] and Free on your plan. We'll also have to fix #65857 so the component knows if it's been purchased yet.

PR Here: #66187

@mreishus mreishus self-assigned this Aug 1, 2022
@mreishus
Copy link
Contributor

mreishus commented Aug 1, 2022

For theme cards in theme showcase, much of this is covered by #65766, which is merged.

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

No branches or pull requests

4 participants