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

[theming] minimap is incorrectly decorated for scm updates (additions, modifications) #6839

Closed
vince-fugnitto opened this issue Jan 7, 2020 · 1 comment · Fixed by #7330
Labels
bug bugs found in the application help wanted issues meant to be picked up, require help scm issues related to the source control manager theming issues related to theming

Comments

@vince-fugnitto
Copy link
Member

Description

The minimap is incorrectly decorated for modified, and added lines (always red).
The minimap should be correctly decorated similarly to the diff-editor.

Single Editor:

editor

Diff Editor:

good

Steps to Reproduce:

  1. open a workspace under source control
  2. ensure that minimap is enabled
  3. perform additions, deletions, modifications
  4. notice that the minimap is incorrectly decorated
@vince-fugnitto vince-fugnitto added bug bugs found in the application theming issues related to theming scm issues related to the source control manager labels Jan 7, 2020
@vince-fugnitto
Copy link
Member Author

It looks like all minimap color decorations do not work at the moment, even during testing of adding pre-existing colors to other decorators (ex: search-in-workspace) did not work successfully.

@akosyakov akosyakov added the help wanted issues meant to be picked up, require help label Jan 7, 2020
vince-fugnitto added a commit that referenced this issue Mar 12, 2020
Fixes #6839

This commit fixes the following problems:
- `overviewRuler` colors were not applied properly for the `scm` extension
- `minimap` colors were not applied properly for the `scm` extension
- colors (`overviewRuler`) are updated to align with vscode

Signed-off-by: Vincent Fugnitto <vincent.fugnitto@ericsson.com>
vince-fugnitto added a commit that referenced this issue Mar 13, 2020
Fixes #6839

This commit fixes the following problems:
- `overviewRuler` colors were not applied properly for the `scm` extension
- `minimap` colors were not applied properly for the `scm` extension
- colors (`overviewRuler`) are updated to align with vscode

Signed-off-by: Vincent Fugnitto <vincent.fugnitto@ericsson.com>
vince-fugnitto added a commit that referenced this issue Apr 24, 2020
Fixes #6839

This commit fixes the following problems:
- `overviewRuler` colors were not applied properly for the `scm` extension
- `minimap` colors were not applied properly for the `scm` extension
- colors (`overviewRuler`) are updated to align with vscode

Signed-off-by: Vincent Fugnitto <vincent.fugnitto@ericsson.com>
vince-fugnitto added a commit that referenced this issue Apr 24, 2020
Fixes #6839

The following commit fixes the following problems:
- fixes the `DecorationOptions#color` to accept `ids` of color variables.
- fixes the decorations provided by `scm` (additions, modifications, removals).
- fixes the color variables for `scm`, aligning the values with vscode.
- fixes the decorations provided by the `siw` (search-in-workspace).

Signed-off-by: Vincent Fugnitto <vincent.fugnitto@ericsson.com>
vince-fugnitto added a commit that referenced this issue Apr 27, 2020
Fixes #6839

The following commit fixes the following problems:
- fixes the `DecorationOptions#color` to accept `ids` of color variables.
- fixes the decorations provided by `scm` (additions, modifications, removals).
- fixes the color variables for `scm`, aligning the values with vscode.
- fixes the decorations provided by the `siw` (search-in-workspace).

Signed-off-by: Vincent Fugnitto <vincent.fugnitto@ericsson.com>
yenteewt added a commit to yenteewt/theia that referenced this issue Apr 27, 2020
Fixes eclipse-theia#6839

The following commit fixes the following problems:
- fixes the `DecorationOptions#color` to accept `ids` of color variables.
- fixes the decorations provided by `scm` (additions, modifications, removals).
- fixes the color variables for `scm`, aligning the values with vscode.
- fixes the decorations provided by the `siw` (search-in-workspace).

Signed-off-by: Vincent Fugnitto <vincent.fugnitto@ericsson.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bugs found in the application help wanted issues meant to be picked up, require help scm issues related to the source control manager theming issues related to theming
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants