Add menu options to open remote in Browser by mangerlahn · Pull Request #67 · gitx/gitx · GitHub
Skip to content

Add menu options to open remote in Browser#67

Closed
mangerlahn wants to merge 3 commits into
gitx:masterfrom
mangerlahn:feature_github_menu
Closed

Add menu options to open remote in Browser#67
mangerlahn wants to merge 3 commits into
gitx:masterfrom
mangerlahn:feature_github_menu

Conversation

@mangerlahn

Copy link
Copy Markdown
Contributor

• Works for the hosts GitHub and Bitbucket
• Added the default shortcut ⇧⌘O
• More can be added easily

• Also: Tweaked the loop which adds the target and ref to context menu
items during creation

• Works for the hosts GitHub and Bitbucket
• Added the default shortcut ⇧⌘O
• More can be added easily

• Also: Tweaked the loop which adds the target and ref to context menu
items during creation
the method remoteUrls is only used internally by the class
@hannesa2

Copy link
Copy Markdown
Contributor

@rjmunro rjmunro left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Could you add gitlab support?

NSString* path = remoteURL.absoluteString;

if ([path containsString:@"github.com"]) {
[[NSWorkspace sharedWorkspace] openURL:remoteURL];

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Will this work for SSH URLs like git@github.com:gitx/gitx.git?

It would be good to have a method that takes a git URL and returns a website URL, then you could easily write unit tests for URLs of different forms.

@hannesa2 hannesa2 added the need rebase The branch needs a rebase to be able to merge label Feb 3, 2021
@hannesa2

hannesa2 commented Mar 1, 2026

Copy link
Copy Markdown
Contributor

@hannesa2 hannesa2 closed this Mar 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

need rebase The branch needs a rebase to be able to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants