-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Add Fig as an installation method to the README #877
Conversation
How's the below diff? I think it better matches the style of the rest of the page. diff --git a/INSTALL.md b/INSTALL.md
index 48b724f..024dbd9 100644
--- a/INSTALL.md
+++ b/INSTALL.md
@@ -75,6 +75,12 @@ This list is incomplete as there are too many
Add `antigen bundle zsh-users/zsh-syntax-highlighting` as the last bundle in
your `.zshrc`.
+#### [Fig](https://fig.io)
+
+Click the `Install Plugin` button on the [Fig plugin page][fig-plugin].
+
+[fig-plugin]: https://fig.io/plugins/other/zsh-syntax-highlighting
+
#### [Oh-my-zsh](https://github.com/robbyrussell/oh-my-zsh)
1. Clone this repository in oh-my-zsh's plugins directory: |
Hey @phy1729 thanks for the quick reply. So, currently https://fig.io/plugins/other/zsh-syntax-highlighting is just a static landing page, however, we are about to push a change that deep links users who hit this page to Fig's desktop app and installs the plugin. So clicking this button will in fact let users instantly download it without an intermediary step. We think the button is more clear and so if possible would love to keep it - also means more downloads for zsh-syntax-highlighting! We have this on a bunch of plugins / frameworks already e.g. https://github.com/sorin-ionescu/prezto |
I'm not fond of the button for a few reasons.
If the URL is changed to start an install rather than just be a landing page the text could be tweaked to indicated such. (Although an HTTP GET request resulting in a plugin getting installed seems against the spirit of the spec.) |
Understood! I'll have Ilkin update it to being hosted on the GitHub CDN using this url
Yes it's a fair point, however, most of the other methods have quite hefty multi-line code blocks. This is the value of Fig - for many, running shell commands is cumbersome and/or scary. Fig makes it easy.
So to really clarify, the button will take you to the plugin splash screen (e.g. https://fig.io/plugins/zsh-syntax-highlighting). We then redirect to a deep link to Fig. If you have Fig installed, the deep link works and we prompt you to confirm installation one last time before it's installed. If you don't have Fig installed, we continue to display the plugin landing page and prompt you to install Fig to set it up. Sorry, not trying to cause confusion here! We made the PR as we have been speaking to @danielshahaf for a while now and his most recent email suggested we should just go ahead and make it! Also, hi Daniel 👋, been a while, hope you are well!! |
This is pure advertisement and lies. The value of this PR is actually negative. Please don't merge it. |
What would be clarified by the button, though? |
I'm not too happy with the image because (1) we try to keep the markdown usable even in source form, not only in rendered form; and (2) no other installation method has anything comparable to Fig's badge, and we don't pick favourites among our downstreams. What might work is to add Fig's logo. Most of the other installation options (all distros and some of the plugin managers) have logos too, so adding logos (to all installation options, not just to Fig) wouldn't break parity. [edit: This is now #902.] |
|
||
Install Zsh Syntax Highlighting in just one click. | ||
|
||
<a href="https://fig.io/plugins/other/zsh-syntax-highlighting" target="_blank"><img src="https://fig.io/badges/install-with-fig.svg" /></a> |
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.
I'd rather remove the target="_blank"
(it's the user's decision, not ours).
Please add alt text to the image, unless the image is removed entirely.
Fig adds apps, shortcuts, and autocomplete to your existing terminal. | ||
|
||
Install Zsh Syntax Highlighting in just one click. | ||
|
||
<a href="https://fig.io/plugins/other/zsh-syntax-highlighting" target="_blank"><img src="https://fig.io/badges/install-with-fig.svg" /></a> |
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.
Fig adds apps, shortcuts, and autocomplete to your existing terminal. | |
Install Zsh Syntax Highlighting in just one click. | |
<a href="https://fig.io/plugins/other/zsh-syntax-highlighting" target="_blank"><img src="https://fig.io/badges/install-with-fig.svg" /></a> | |
Click [here](https://fig.io/plugins/other/zsh-syntax-highlighting) to 1-click install `zsh-syntax-highlighting` using [Fig](https://fig.io) |
@phy1729 @danielshahaf - My apologies for massive delay here (have just been cleaning out my GH notifications) I've updated the PR factoring in you commentary above. Totally understand the concerns about the button, I've removed it. Intention is not to be markety or salesy. We truly do have a bunch of users, z-sy-h is incredibly popular (we feature it heavily in our store), and having us listed as an install method helps other users quickly download it. Apologies again for the issues above - please let me know if any more questions/concerns! Brendan |
The Fig Plugin Store supports 1-click install for 400+ shell plugins. We have over 100k users, thousands of whom manage their shell configuration with Fig.
Zsh Syntax Highlighting is already listed in the store so we'd love to have it listed as a download method on your readme.
Thanks so much and please let me know if you have any questions!