search

SAP / ui5-language-assistant

VSCode Extension and Editor Tooling for SAPUI5
Apache License 2.0
54 stars 26 forks source link

Add warning if property contains the default value #377

Open ghost opened 3 years ago

ghost commented 3 years ago

Hello, I have a feature request for you:

I sometimes find myself copying source code from the samples page. These examples are sometimes "bloated" (e.g. SimpleForm). Or I just try different values for a specific property and then stick with the default value because I like it the most (e.g. different backgroundDesigns for a Panel.

It would be nice to know that I can just remove the property because it contains the default value.

bd82 commented 3 years ago

Hello @OTTOxMarc

I think that such a feature would have a pre-requirement of "user configurable validation rules" capability first. Because this is basically a style preference, and we cannot always show "Style" warnings, particularly if there are many of them.

ghost commented 3 years ago

Yes, I see that this might not be important for everyone and should be (de)activatable in some kind of user settings.