-
Notifications
You must be signed in to change notification settings - Fork 40
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 more fork options #21
Comments
Thanks for the suggestions. Omnibox is a great extension! :) Responses inline.
I am ambivalent about the merge. With recent changes to Chrome, the extensions have become obnoxiously visible and it is better to keep them to a minimum. However, on the other hand, I would like to maintain Firefox compatibility. So I would wait for a bit to see how the extension ecosystem evolves. Or, we could attempt to abstract out the functioning and keep it at both places. Suggestions?
The UI was the primary problem, though I've found that the community usually converges to only one maintained fork. I like your suggestion about creating a small hover card showing the other forks in order of stargazers. However, getting the recency information about those other forks leads me to the second point.
At the moment, I am using Github's compare API to determine if the Additionally, Lovely Forks does not, at the moment, allow for Github authentication. It uses two API calls for each Github page (and caches results): one to get the list of forks ordered by What did you mean by "x/y"? |
Owner/repo |
The two extensions are completely unrelated. You can merge efforts by cross-contributing but it doesn't make sense to merge just to save an icon (which can be hidden anyway)
If you want to see more forks, you can click the fork button or add a link to the members page |
Now there is useful-forks.github.io to solve the same problem. |
I've been wanting to add something like this for a while to my Github Omnibox extension any interest in merging efforts?
Anyway, I think it'd be nice if you didn't limit suggestions to only the primary fork, but continue showing forks everywhere. Also, I might change the text and suggestion to be a bit more informative like "also a popular fork: x/y" for the fork with the second-highest stars and "also an active fork: x/y" for the one with the most recent commit(s). It would be nice if it could also count the number of commits. The flame icon doesn't really illustrate much. You would be able to combine them asl: "also a popular active fork: x/y"
I think it'd also be cool if it was a dropdown (on hover) maybe triggered by an ellipses or down arrow that lets you see the top 3-5 most popular / active forks?
My chrome extension adds a dropdown so it's not too difficult:
The text was updated successfully, but these errors were encountered: