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

[css] Redundant css suggestion for color #43464

Closed
ramya-rao-a opened this issue Feb 12, 2018 · 6 comments
Closed

[css] Redundant css suggestion for color #43464

ramya-rao-a opened this issue Feb 12, 2018 · 6 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug css-less-scss Issues and items concerning CSS,Less,SCSS styling verified Verification succeeded
Milestone

Comments

@ramya-rao-a
Copy link
Contributor

  • VSCode Version: Any
  • OS Version: Any

Steps to Reproduce:

  1. Use the below content and trigger suggestions at the end of the property value
.foo {
   color: #123
}
  1. The suggestion list has 2 entries. 1 from emmet, and the other from the css extension repeating the value that is already typed.

screen shot 2018-02-11 at 8 25 55 pm

Does this issue occur when all extensions are disabled?: Yes

@vscodebot vscodebot bot added the css-less-scss Issues and items concerning CSS,Less,SCSS styling label Feb 12, 2018
@aeschli aeschli added bug Issue identified by VS Code Team member as probable bug css-less-scss Issues and items concerning CSS,Less,SCSS styling and removed css-less-scss Issues and items concerning CSS,Less,SCSS styling labels Feb 16, 2018
@aeschli aeschli added this to the Backlog milestone Feb 28, 2018
@praveenpuglia
Copy link
Contributor

Is this part of the css-language-service?

@aeschli
Copy link
Contributor

aeschli commented Mar 1, 2018

@praveenpuglia Yes, but it's ok to have the issue here.

@aeschli aeschli modified the milestones: Backlog, March 2018 Mar 7, 2018
@octref
Copy link
Contributor

octref commented Mar 23, 2018

@ramya-rao-a What are you asking for?
Suggesting #123 is working as expected from css. If you want exact match to come before others there are issue for it but I can't find.

@octref octref added info-needed Issue requires more information from poster and removed bug Issue identified by VS Code Team member as probable bug labels Mar 23, 2018
@ramya-rao-a
Copy link
Contributor Author

ramya-rao-a commented Mar 23, 2018

@octref My question was around why is the exact match even suggested in the first place, it doesnt add any value. I tried the same in other languages. Looks like this is an issue with the filtering logic in the suggestions in VS Code.

When I was trying to integrate emmet in the css extension, this issue was more pronounced as numbers were trigger characters. We can close this issue as we no longer have the integration and also because this is not due to the css extension, but is due to the filtering

@octref
Copy link
Contributor

octref commented Mar 23, 2018

@ramya-rao-a Well just found out I was the guy that opened the exact issue: #21362

@aeschli
Copy link
Contributor

aeschli commented Mar 26, 2018

IMO @ramya-rao-a's request is valid here.
If there's a known set of values for a css property, e.g. for display, we should show each value as a proposal regardless of what is already typed.
But, In this case, we guessed possible values by looking at other locations. When doing that we should ignore the current value. But, if some other location has that value as well, then it makes sense to show the color value.

@aeschli aeschli reopened this Mar 26, 2018
@aeschli aeschli changed the title Redundant css suggestion for color [css] Redundant css suggestion for color Mar 26, 2018
@aeschli aeschli added bug Issue identified by VS Code Team member as probable bug and removed info-needed Issue requires more information from poster labels Mar 26, 2018
@mjbvz mjbvz added the verified Verification succeeded label Mar 27, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators May 10, 2018
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 css-less-scss Issues and items concerning CSS,Less,SCSS styling verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

5 participants