Skip to content
This repository was archived by the owner on Oct 4, 2021. It is now read-only.

Conversation

@ketttY
Copy link
Contributor

@ketttY ketttY commented Dec 10, 2019

  • For linked files only provide "Remove" (bound with cmd + delete shortcut) like VS Win

Screen Shot 2019-12-10 at 2 23 57 PM

VS Win:

Unknown

Fixes: https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1026629/

@mrward
Copy link
Contributor

mrward commented Dec 10, 2019

Still seems to be some discussion from the design team about the behaviour here - I guess after this code change was made.

@ketttY
Copy link
Contributor Author

ketttY commented Dec 10, 2019

@mrward Oops... Just saw that :) Thanks for the heads-up, adding do-not-merge label for now

Copy link
Member

Choose a reason for hiding this comment

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

This text needs to be localized

@ketttY
Copy link
Contributor Author

ketttY commented Dec 13, 2019

@monojenkins rebase

@monojenkins monojenkins force-pushed the vsts-1026629-delete-command-for-linked-files branch from fecf13d to 28b5155 Compare December 13, 2019 14:08
@ketttY ketttY merged commit 6c67e39 into master Dec 16, 2019
@Therzok Therzok deleted the vsts-1026629-delete-command-for-linked-files branch December 17, 2019 09:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants