-
Notifications
You must be signed in to change notification settings - Fork 48
Conversation
License: MIT Signed-off-by: Henrique Dias <hacdias@gmail.com>
License: MIT Signed-off-by: Henrique Dias <hacdias@gmail.com>
content/post/057-ipfs-desktop-0-9.md
Outdated
|
||
![Experiments section](/057-ipfs-desktop-0-9/npm-ipfs.png) | ||
|
||
**Please note** that this feature is intended for those who have Node.js installed on their system. Moreover, the activation will only work if your setup does not require additional permissions for executing `npm install -g`. |
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.
is there anything folks can do if their setup DOES require additional permissions, or are they SOL?
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.
On that case, we recommend them to execute the command themselves (npm i -g npm-ipfs
). That's the message we show if it fails.
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.
Can you link Node.js to its install page?
This is awesome!!! Great initiative to write this up. Made some suggestions and nits, but love the direction you're heading. =] |
Co-Authored-By: MollyM <momack2@users.noreply.github.com>
Co-Authored-By: MollyM <momack2@users.noreply.github.com>
Co-Authored-By: MollyM <momack2@users.noreply.github.com>
Co-Authored-By: MollyM <momack2@users.noreply.github.com>
Co-Authored-By: MollyM <momack2@users.noreply.github.com>
Co-Authored-By: MollyM <momack2@users.noreply.github.com>
Co-Authored-By: MollyM <momack2@users.noreply.github.com>
@momack2 thanks a lot for the review! I was just waiting to release some bug fixes to write a post and so here we are 😄 |
Co-Authored-By: MollyM <momack2@users.noreply.github.com>
License: MIT Signed-off-by: Henrique Dias <hacdias@gmail.com>
License: MIT Signed-off-by: Henrique Dias <hacdias@gmail.com>
License: MIT Signed-off-by: Henrique Dias <hacdias@gmail.com>
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.
Looks fantastic, great idea to cover some past/present/future! There are a few grammar and typo fixes in the review, and some suggestions which you can take or leave as you prefer, but 👍🏼 from me!
content/post/057-ipfs-desktop-0-9.md
Outdated
- **Handle `ipfs://`, `ipns://` and `dweb:` links**. If you have IPFS Desktop installed and you click on [ipns://ipfs.io](ipns://ipfs.io), that request will go through be redirected to your own gateway via the app, or to the public gateway if yours is not online. | ||
- **Easily add files to IPFS**. Just drag and drop them to the application icon, either on the menubar for macOS users, of the shortcut icon for Windows users. | ||
- **Add screenshots to IPFS**. You can click on 'Take Screenshot' on the app's menu or enable the global shortcut on "Settings". After taking a screenshot, a shareable link will be copied to your clipboard! | ||
- **Download any CID content**. If you would like to download the contents of some CID or IPFS/IPNS path, you can copy it, go to the dropdown menu and select 'Download Hash'. Then, just pick wherever you want to write those files to! This is also available through a global shortcut. |
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.
Can you note what the global shortcut is for each OS?
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.
Hmmm... I wonder if it is really necessary to have on the blog post. I am saying this because these are features that are required to be manually enabled. Thus, the user will always see the shortcuts on settings
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.
Awesome update! Thank you for taking time to write this 👍
Small suggestions below and inline
Digression on image size
Images are huge (each PNG is 0.5MB), we could make it easier for mobile users by scaling them down or optimizing via https://squoosh.app.
Something to consider is to switch to WebP (supported by both Chromium and Firefox), which supports both lossy compression and retains transparency of pretty mac screenshots. Did a test run on one of screenshots and the size decreased by 75% without a visible quality degradation:
Co-Authored-By: Marcin Rataj <lidel@lidel.org>
As @lidel said, I don't mind converting all images to webp. We will get much smaller image files :) Otherwise I can just convert them into jpegs with white background. |
I've just converted all images into jpegs for the sake of compatibility with browsers other than Firefox and Chrome. @autonome perhaps not including GIFs is the best thing to do also due to their size... Anything else I should do before merging this? |
Updated the post number since I messed you up. I also edited the URL to reflect the date rather than the post number to match the pattern folks seem to be following. You'll need to update the date in the YAML data and URL fields again when you're ready to merge.
@hacdias I just submitted a commit updating your post number to 058 since I just published as 057. (Not sure which images are new and should have this change made and which are pulled from older posts.) I also updated your YAML to reflect today's date in the |
@terichadbourne thanks! Just updated the image directory and updated the date to tomorrow. Tomorrow will be merging if I don't get any objections until then :) |
This is a draft of a post highlighting the new features of IPFS Desktop and some other stuff. I'll be updating this really soon
License: MIT
Signed-off-by: Henrique Dias hacdias@gmail.com