-
Notifications
You must be signed in to change notification settings - Fork 128
Conversation
layouts/partials/install.html
Outdated
<h5>Is the command line not your jam? Download <a href="https://github.com/ipfs-shipyard/ipfs-desktop">ipfs-desktop</a>!</h5> | ||
</div> | ||
</div> | ||
</section> |
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.
There seems to be some indentation inconsistency here as the final element is out of alignment with the opening
Looked through he code and it seems fine except for a few comments I’ve made, but I’m not sure what the information architecture plan is for this page so someone else will need to review for final signoff. |
I like it. Let's fix the awkward gradient clash where the install section meets the implementations section. In the current version, it's not clear that "install ipfs" is a link... ...in fact, i'd expect an install link to take me to the instructions to install it, rather than what to do after installing it. The big problem here is that the actuall install instuctions are too long / awkward. Ideally we'd detect your OS and say something simpler like: # Install the go-ipfs command line tool from homebrew
brew install ipfs
# Initialise your repo in ~./ipfs and create your unique Peer ID
ipfs init
# Fetch the README, peer-to-peer style!
ipfs cat /ipfs/QmYwAPJzv5CZsnA625s3Xf2nemtYgPpHdWEz79ojWnPbdG/readme Prefer to install it manually? Check out the install docs The command line not your jam? Download IPFS Desktop i've been using brew to install ipfs on new machines, and it's been up to date, and no hassle. The alternatives are:
|
@olizilla The problem with |
oh no, i don't want to do that, I was suggesting
so, only suggest homebrew if your on mac or linux. we could suggest chcolatey on windows (have you tried out installing go-ipfs via choccy @hacdias ?) |
The most important point that I'm looking for agreement on is... We need a blessed, 1 command install process for the ipfs command line tools, so we can show it off right on the homepage. I suspect there will be some debate on what that looks like, but we should pick one. Explaining that they should download the download the tar and add it to their PATH is way too technical for folks that don't know what their PATH is, and at the same time totally redundant for folks that do know. Another idea is that we could flip the install section to recommend the desktop app first, for casual users, so that we get folks to install the auto-updating, home user optimised app, rather than the command line tool which, out of the box will open many hundreds of connections. |
That being said, one could argue if they don't want to download the tarball and add it to their PATH, they could use |
wireframe proposal for how we can frame the install step on the homepage... basically recommend package managers for most platforms, and link out the manual install process for experts and the curious. The link to "go install desktop" should also be offered there too, as it is in the current PR We can make an educated guess to what platform the user is on from js and flip the active tab accordingly |
Closing this since we're addressing this via various other means, particularly including https://github.com/ipfs/docs/issues/188 . |
In an effort to make it easier for users to quickly get started with IPFS, I added an
Install
section aboveImplementations
. TheInstall
section shows users where they can download IPFS and a list of basic CLI commands. This PR addresses #293, #227 #242Debating if I should add an ipfs-companion link next to ipfs-desktop.