Asset store item has "Resources" folder containing UI buttons for inspector GUIs

I'm using one asset that a class which generates an inspector GUI.

public class Generator_Inspector: Editor {
...
Texture _saveIcon = Resources.Load("UI/save@0.5x", typeof (Texture)) as Texture;
}

Am I wrong in thinking that these UI button images should not be in a folder called "Resources"?
They would be included in the build, wouldn't they?

 

I think if it's under an Editor folder as well (can't tell from that code alone) then it won't be included in a build. Still, I'm 99% certain that there's a documented way to load icons in the Editor, and that isn't it, and the documented best practices are pretty clear about not using Resources folders, so I agree this stuff shouldn't be let through to the Asset Store.

 

Resources folder for editor stuff is just fine, really the most practical way to do it. As the penguin said, it should be Editor/Resources to exclude from build. If it’s not you can move it there.

 

I believe EditorGUIUtility.Load(...) is what I was thinking of.