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

Add getting started button #911

Merged
merged 1 commit into from
Feb 3, 2021
Merged

Conversation

gtsiolis
Copy link
Contributor

@gtsiolis gtsiolis commented Dec 17, 2020

This will add a secondary button to link to the new Getting Started section in the docs which mentions how you can use Gitpod.

Context: This builds on top of the original proposal in https://github.com/gitpod-io/website/issues/882#issue-759375835 and moves us one step closer to https://github.com/gitpod-io/website/issues/876.

BEFORE AFTER
image image

@gtsiolis
Copy link
Contributor Author

@JohannesLandgraf WDYT? Feel free to close this if you think this is going into the wrong direction. /cc @ChristinFrohne

@@ -78,6 +78,9 @@ const Banner = () => (
<a href="#get-started" className="btn btn--cta">
Try Now
</a>
<a href="https://www.gitpod.io/docs/getting-started/" className="btn" style={{ marginLeft: '8px' }}>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

thought: Alternatively, we could link to the Introduction section, it just seemed natural and more practical to link to the Getting Started section. /cc @JohannesLandgraf

Copy link
Contributor

Choose a reason for hiding this comment

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

Agree, also the Getting Started Section links back to the Introduction Section so should be fine.

@chrifro
Copy link
Contributor

chrifro commented Dec 17, 2020

I as a user would be confused about the differences between try now and get started. Instead, we could add the link to the documentation to https://www.gitpod.io/#get-started

Copy link
Contributor

@JohannesLandgraf JohannesLandgraf left a comment

Choose a reason for hiding this comment

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

I really like the idea @gtsiolis, let's merge after you resolved the conflicts

@gtsiolis gtsiolis force-pushed the gt/add-getting-started-button branch from 175e0bd to 824e20e Compare February 3, 2021 12:13
@gtsiolis
Copy link
Contributor Author

gtsiolis commented Feb 3, 2021

Merging per https://github.com/gitpod-io/website/pull/911#pullrequestreview-582184591. Thanks @JohannesLandgraf @ChristinFrohne for the feedback! 🙏

@ChristinFrohne regarding https://github.com/gitpod-io/website/pull/911#issuecomment-747457299, this will probably become better and more clear after the redesign but I'd suggest to avoid removing the only primary action that prompts the user to immediately open a new environment using an example project from this section. Let's discuss this in a new issue if you think it's worth putting more effort in this before the redesign. 🌱

@gtsiolis gtsiolis merged commit 2e6934c into master Feb 3, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants