Skip to content

Share and Copy As consolidation #157722

Description

@kieferrm

Version: 1.71.0-insider
Commit: bcb31b9
Date: 2022-08-09T17:04:54.174Z
Electron: 19.0.11
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: 10.2.154.15-electron.0
OS: Darwin arm64 21.6.0
Sandboxed: Yes

As well as 1.70 stable.

The current state:

  1. Share is only available in the File Menu.

image

  1. Copying file URLs for GH is available in the edit menu under Copy As...

image

  1. The explorer context menu inlines the Copy file URLS for GH actions, but doesn't have Share.

image

  1. The editor context menu has "Share" and "Copy As" submenus.

image

I think there is need for consolidation here:

  1. The explorer and the editor context menu should be aligned on "Share" and "Copy As"
  2. In both context menus "Share" share should take the place of "Copy As". I personally don't see any good reason to have both and "Share" is closer to the goal users pursue.
  3. The consolidated "Share" should be in the File or Edit menu (but not in both).

Activity

  1. chrisdias commented on Aug 10, 2022

    @chrisdias
    Contributor

    In a nutshell, the Share menu should look like this?

    Share >
      Copy vscode.dev link
      Copy GitHub Permalink
      Copy GitHub Permalink as Markdown
      Copy GitHub Head Link
    

    Should we have Copy GitHub Head Link as Markdown as well? How often are these used?

    I think it would be great to also include the Publish to GitHub command here as well if not backed by a GitHub repo already.

  2. alexr00 commented on Aug 10, 2022

    @alexr00
    Member

    Maybe we should just rename the "Copy As" menu to "Share" if we don't want to have both?

  3. kieferrm commented on Aug 10, 2022

    @kieferrm
    MemberAuthor

    Maybe we should just rename the "Copy As" menu to "Share" if we don't want to have both?

    💯% yes. That's the proposal.

  4. kieferrm commented on Aug 10, 2022

    @kieferrm
    MemberAuthor

    And when I use GH Remote Repositories on desktop I don't have "Share" at all, and I don't have any way to easily get the corresponding vscode.dev url for the file I look at. Joyce Er (@joyceerhl)

  5. alexr00 commented on Aug 11, 2022

    @alexr00
    Member

    We currently rely on the built in git extension to understand whether there are repositories. We have a couple options if we want to support share where we don't have information from the git extension:

    • Have extension API that abstracts away where the repositories come from. This is how GitHub Pull Requests and Issues works. If we go this route, maybe we remove all that abstraction away from GHPRI and put it the built in GitHub extension. Then GHPRI can take a dependency on the GitHub extension.
    • Or, Remote Hub can contribute the Copy vscode.dev Link command.

    The first option sounds like a good longer term goal.

  6. joyceerhl commented on Aug 11, 2022

    @joyceerhl
    Contributor

    RemoteHub can contribute the Copy vscode.dev Link command in the short term to align the web and desktop experiences. Alex Ross (@alexr00) I will submit PRs for that (requires proposed API access).

  7. alexr00 commented on Sep 1, 2022

    @alexr00
    Member

    After some further discussion, we won't do the GHPRI/GitHub/git refactor at this time as the payoff doesn't seem worth it. That's now tracked here: #159763

    I will still work on the consolidation of the menus in September.

  8. alexr00 commented on Sep 21, 2022

    @alexr00
    Member

    Not all the extensions that currently contribute to the "Copy As" menu aren't really contributing "Share" commands. Here's what the "Copy As" menu looks like with Gitlens installed:

    image

    "Copy SHA" and "Copy Message" aren't related to sharing. Arguably, the "Copy GitHub Permalink as Markdown" command is also not "Share". I think we still need the "Copy As" menu.

    I'll move all the "Copy GitHub" commands into the "Share" menu though.

  9. alexr00 commented on Sep 21, 2022

    @alexr00
    Member

    Share menu with these changes:

    image

    Which, with the extensions I have installed, leaves me with this in the Copy As menu:

    image

  10. added
    bugIssue identified by VS Code Team member as probable bug
    on Sep 21, 2022
  11. alexr00 commented on Sep 22, 2022

    @alexr00
    Member

    Summarizing some team discussion:

    • "Copy As" makes more sense since all the commands are copy commands.
    • "Share" sounds like something that should have more than just "Copy" commands.
    • Most of the team feels that "Share" is not needed if we have "Copy As".
    • On the other hand, some folks didn't think to look for the permalink commands under "Copy As" because they sound more like "Share" commands.

    Our goal with introducing the "Share" entry point was to make it easy for users to find the "Copy vscode.dev Link" command. But I still don't think that we should simply rename the whole "Copy As" menu to "Share". I'm inclined to keep the changes in #157722 (comment), but only if we're confident that we won't change them again next iteration.

  12. locked and limited conversation to collaborators on Nov 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

bugIssue identified by VS Code Team member as probable bugverifiedVerification succeeded

Type

No type

Projects

No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions