-
Notifications
You must be signed in to change notification settings - Fork 8
Pricing page template #34
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
Pricing page template #34
Conversation
nienn
commented
Jan 20, 2022
- Merge after Product page template #28
This was removing images as well, which we don't want.
|
|
||
| <div class="description"> | ||
|
|
||
| <div class="ui list relaxed divided"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we mimic above and instead use "All open source plan features, plus:" here? The only difference is ads are removed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was done in commit 41dcffd.
Co-authored-by: Anthony <[email protected]>
Co-authored-by: Anthony <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some last review for this first. Updates look good so far!
| <div class="ui vertical segment basic very padded center aligned"> | ||
| <h3 class="ui header massive">Enterprise plans</h3> | ||
| <p> | ||
| Offer flexible options for your organizations, whether hosting public or private documentation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking again, this sentence can be dropped. The lead we want for enterprise plans is probably the line below this, "For enterprise documentation needs..."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Following discussion above, this section was significantly reduced in commit 3a673c9 and the text substituted by a link to the docs page choosing-between-our-two-sites and one short paragraph pulled from that page.
| We are happy to work with you on any additional requests you might have. Reach out with your specific needs. | ||
| </p> | ||
| <p> | ||
| Still Have Questions? <span class="ui segment basic tiny"><a href="" class="ui button black mini">Contact us</a></span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Still Have Questions? <span class="ui segment basic tiny"><a href="" class="ui button black mini">Contact us</a></span> | |
| Still have questions? <span class="ui segment basic tiny"><a href="mailto:[email protected]" class="ui button black mini">Contact us</a></span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure why this wasn't committed with the batch but the next commit - 3a673c9 - reduces this section so it no longer applies.
Co-authored-by: Anthony <[email protected]>