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

diffEditor.renderIndicators doesn't work for peek #123165

Closed
svipas opened this issue May 6, 2021 · 2 comments
Closed

diffEditor.renderIndicators doesn't work for peek #123165

svipas opened this issue May 6, 2021 · 2 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders scm General SCM compound issues verified Verification succeeded
Milestone

Comments

@svipas
Copy link
Contributor

svipas commented May 6, 2021

  • VS Code Version: 1.56.0
  • OS Version: macOS

Steps to Reproduce:

  1. "diffEditor.renderIndicators": false
  2. Open folder with Git repository and change something
  3. Open peek git diff and you will see + and - indicators

This issue was fixed in 2018 by @joaomoreno, but seems like it still exists or maybe it's regression. Old issue: #50362

@joaomoreno joaomoreno assigned rebornix and unassigned joaomoreno May 10, 2021
@rebornix rebornix added bug Issue identified by VS Code Team member as probable bug diff-editor Diff editor mode issues labels Jun 4, 2021
@rebornix rebornix added this to the June 2021 milestone Jun 4, 2021
@rebornix rebornix modified the milestones: June 2021, July 2021 Jul 1, 2021
@rebornix rebornix modified the milestones: July 2021, August 2021 Jul 28, 2021
@rebornix rebornix modified the milestones: August 2021, September 2021 Aug 22, 2021
@rebornix rebornix modified the milestones: September 2021, On Deck Sep 29, 2021
@alexdima
Copy link
Member

@joaomoreno I could not find any hard-coding of renderIndicators: false in dirtydiffDecorator.ts as hinted in #50362 . If we would want to hard-code that to false, I think we would do that here. But hard-coding that to renderIndicators: false might mean that the embedded diff editor is no longer accessible as it would rely only on colors to convey diff information.

@svipas I've also noticed that simply configuring "diffEditor.renderIndicators": false is respected correctly in the embedded diff editor, so given we have that setting, I am not sure why we would want to hard-code them to be hidden in the embedded diff editor.

@alexdima alexdima assigned joaomoreno and unassigned rebornix Oct 15, 2021
@alexdima alexdima removed the diff-editor Diff editor mode issues label Oct 15, 2021
@alexdima alexdima removed this from the On Deck milestone Oct 15, 2021
@alexdima alexdima removed the bug Issue identified by VS Code Team member as probable bug label Oct 15, 2021
@joaomoreno
Copy link
Member

I've went along and disabled them statically.

@joaomoreno joaomoreno added this to the October 2021 milestone Oct 25, 2021
@joaomoreno joaomoreno added bug Issue identified by VS Code Team member as probable bug scm General SCM compound issues labels Oct 25, 2021
@mjbvz mjbvz added the verified Verification succeeded label Oct 27, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Dec 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders scm General SCM compound issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

6 participants
@joaomoreno @rebornix @alexdima @svipas @mjbvz and others