Skip to content
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

Update README Bookmarklet #173

Merged
merged 2 commits into from
Feb 24, 2021
Merged

Update README Bookmarklet #173

merged 2 commits into from
Feb 24, 2021

Conversation

shanyuhai123
Copy link
Contributor

Update Bookmarklet in README. Make the URL recoverable to github.com.

Update Bookmarklet in README. Make the URL recoverable to github.com.
@vercel
Copy link

vercel bot commented Feb 23, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/conwnet/github1s/3RcsUBkx53wr9Uik779jENahEUgM
✅ Preview: https://github1s-git-fork-shanyuhai123-shanyuhai123-patch-1-conwnet.vercel.app

@shanyuhai123 shanyuhai123 changed the title Update README.md Update README Bookmarklet Feb 23, 2021
xcv58
xcv58 previously approved these changes Feb 23, 2021
@@ -19,7 +19,7 @@ For browser extensions, see [Third-party Related Projects](https://github.com/co
Or save the following code snippet as a bookmarklet (GitHub markdown doesn't allow js links, so just copy it into a bookmark).

```
javascript: window.location.href = window.location.href.replace('github.com', 'github1s.com')
javascript: window.location.href = window.location.href.replace(/github(1s)?.com/, function(match, p1) { return p1 ? 'github.com' : 'github1s.com' })
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could you please add an explanation for the behavior to convert back to github.com?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Convert back to github.com to view issues and switch branch(Although mentioned in #155, I haven't seen it before).

Copy link
Collaborator

Choose a reason for hiding this comment

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

I mean add this in the README file, thanks.

Update Bookmarklet in README.
You can use it to quickly switch between github.com and github1s.com.
Copy link
Owner

@conwnet conwnet left a comment

Choose a reason for hiding this comment

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

lgtm, thanks

@conwnet conwnet merged commit fb730bc into conwnet:master Feb 24, 2021
conwnet pushed a commit that referenced this pull request Feb 27, 2021
* Update README.md

Update Bookmarklet in README. Make the URL recoverable to github.com.

* Update README Bookmarklet

Update Bookmarklet in README.
You can use it to quickly switch between github.com and github1s.com.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants