Add more native properties

[abdonrd]

There is the no-native-attributes rule:

https://github.com/43081j/eslint-plugin-lit/blob/master/docs/rules/no-native-attributes.md

But we also want to avoid native properties like className:

https://developer.mozilla.org/en-US/docs/Web/API/Element/className

We should have a different rule?
Or just add more elements to the array?

eslint-plugin-lit/src/rules/no-native-attributes.ts

Lines 11 to 44 in 7e40fca

	const NATIVE_ATTRS = [
	'accesskey',
	'autocapitalize',
	'autofocus',
	'class',
	'contenteditable',
	'contextmenu',
	'dir',
	'draggable',
	'enterkeyhint',
	'exportparts',
	'hidden',
	'id',
	'inert',
	'inputmode',
	'is',
	'itemid',
	'itemprop',
	'itemref',
	'itemscope',
	'itemtype',
	'lang',
	'nonce',
	'part',
	'popover',
	'role',
	'slot',
	'spellcheck',
	'style',
	'tabindex',
	'title',
	'translate',
	'virtualkeyboardpolicy'
	];

Thanks!

[43081j]

sorry for the slow response, have been away the last few weeks

you've got a very good point.

it'd be so easy to just add them to the list, but i do feel like the right thing to do is probably two rules:

• no-native-attributes
• no-native-properties

basically identical code other than the list of names.

what do you think?

[abdonrd]

I have the same feelings!