modern-normalize icon indicating copy to clipboard operation
modern-normalize copied to clipboard

vs code gives warning about modern-normalize

Open mu-us61 opened this issue 3 years ago • 4 comments
trafficstars

I dont know if this is really an issue or not, but nonetheless kind of distracting warnings

image

mu-us61 avatar Nov 12 '22 18:11 mu-us61

Not an issue. Checking the VSCode documentation, you could to turn off CSS validation for your workspace in order to not get these warnings anymore: https://code.visualstudio.com/Docs/languages/CSS#_syntax-verification-linting

scheinercc avatar Jan 05 '23 03:01 scheinercc

Off course there are non standard properties... This is what this library is about.

More distracting is that this issue is open for a half a year.

Emkas avatar Apr 20 '23 08:04 Emkas

Is it okay to add the standard properties VS is asking for with the same value as the one highlighted? I did that with mine and it stopped giving me warnings. VS told me to add the standard property as to maintain compatibility. This may even be added to the code in the future. I'm just a newbie though. Hope to get some feedback.

This was the change I did.

button,
[type='button'],
[type='reset'],
[type='submit'] {
	-webkit-appearance: button;
	appearance: button; /* Added line */
}

[type='search'] {
	-webkit-appearance: textfield; /* 1 */
	appearance: textfield; /* Added line*/
	outline-offset: -2px; /* 2 */
}

ghost avatar Jul 29 '23 05:07 ghost

The idea is to use the non-standard properties to fix stuff only on the browsers that need fixing. That is why they are not using the standard properties.

Just ignore the VS errors or exclude this file from VS checking. These non-standard properties are intentional and quite frankly the whole point of these libraries.

Rafagd avatar Aug 17 '23 13:08 Rafagd