Skip to content
This repository was archived by the owner on Sep 11, 2025. It is now read-only.

Update pricing page 2#70

Merged
dadlerj merged 18 commits intomasterfrom
da/update-pricing
Jan 15, 2019
Merged

Update pricing page 2#70
dadlerj merged 18 commits intomasterfrom
da/update-pricing

Conversation

@dadlerj
Copy link
Member

@dadlerj dadlerj commented Jan 3, 2019

@francisschmaltz thoughts on the latest update here? Replaces #66

image
image
image

@francisschmaltz
Copy link
Contributor

@dadlerj Looks good! I made some small tweaks to how the site behaves on mobile so just make sure those are included (it looks like they are).

@dadlerj dadlerj mentioned this pull request Jan 14, 2019
@dadlerj
Copy link
Member Author

dadlerj commented Jan 15, 2019

This latest update changes some wording and ordering to make things slightly more logical in grouping.

E.g.:

  • SSO wording is now a bit more clear
  • user limits bullets are now grouped with SSO
  • support bullets show up at the bottom of the list
  • removed some obvious/unnecessary bullets (e.g. "no user limit" on Enterprise)
  • added hyphens where necessary

image

Copy link
Contributor

@francisschmaltz francisschmaltz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Contributor

@francisschmaltz francisschmaltz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The bottom buttons need to be block links on small viewport.

Copy link
Contributor

@francisschmaltz francisschmaltz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated contact buttons with be full width on mobile to match the rest of the page.

@dadlerj
Copy link
Member Author

dadlerj commented Jan 15, 2019

Interestingly enough, they already appear full width before your edit to me:
image

Anything thoughts?

@francisschmaltz
Copy link
Contributor

It was the very bottoms links that say "Deploy" and "Contact us"

@dadlerj
Copy link
Member Author

dadlerj commented Jan 15, 2019

Ah, thanks

@dadlerj dadlerj merged commit e4ab2e6 into master Jan 15, 2019
@dadlerj dadlerj deleted the da/update-pricing branch January 15, 2019 16:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants