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

Prompt shows when pressing "accept merge" #153615

Closed
connor4312 opened this issue Jun 28, 2022 · 6 comments
Closed

Prompt shows when pressing "accept merge" #153615

connor4312 opened this issue Jun 28, 2022 · 6 comments
Assignees

Comments

@connor4312
Copy link
Member

connor4312 commented Jun 28, 2022

Testing #151024

make sure the message doesn't show when pressing 'Accept'

This doesn't appear to work for me, I get the confirmation prompt both with autosave on and off:

Kapture.2022-06-28.at.15.38.48.mp4
@connor4312 connor4312 changed the title Prompt shows when pressing "accept changes" Prompt shows when pressing "accept merge" Jun 28, 2022
@jrieken jrieken added *duplicate Issue identified as a duplicate of another issue(s) merge-editor labels Jun 29, 2022
@jrieken
Copy link
Member

jrieken commented Jun 29, 2022

This is a duplicate. Unfortunately, I don't recall the exact issue number nor its title 😳 Anyways, searching for issues with the same labels will likely help you 👉
https://github.com/microsoft/vscode/issues?utf8=✓&q=is%3Aopen+label%3Amerge-editor

@jrieken
Copy link
Member

jrieken commented Jun 29, 2022

#153492

@vscodenpa
Copy link

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.

Happy Coding!

@jrieken jrieken removed the *duplicate Issue identified as a duplicate of another issue(s) label Jun 29, 2022
@jrieken jrieken reopened this Jun 29, 2022
@jrieken
Copy link
Member

jrieken commented Jun 29, 2022

I have fixed #153492 but the dialog still shows - despite the pressing Accept Merge. There is technical challenges but I can also sell this as feature: When pressing accept in the presence of conflicts you get a prompt.

@connor4312 How does that sound? Do you agree with marking this as-designed?

@connor4312
Copy link
Member Author

🤷 I can see that working, we can see if we get feedback on it whether it's confusing for users.

@connor4312
Copy link
Member Author

Gah, Github changing their cmd+enter keybinding has caught me five times already today 😠

@connor4312 connor4312 reopened this Jun 29, 2022
@jrieken jrieken closed this as completed Jun 30, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Aug 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants