Closed HookyQR closed 8 years ago
Of course, return
would do. Think I was being a little over explicit there.
Thanks for reporting, that was indeed a bug. I've only changed the extension to the fileName. Limiting the icon to certain extensions sounds useful, though I'm not yet sure what the best design would be. I might implement such feature later.
I wanted to change the javascript icon to the nodejs_small icon and discovered that this caused the icon to show up as the main and the secondary icon when secondaries were turned on. I've tracked down the problem. The
UserDIctionary,findFIleName
function ignores thefileName
parameter. This can be fixed by changing the body to the following code:This has the handy side effect that you can specify secondary icons for specific files, just as are done in the presets, such as:
With the second line as it is, any filename (of any extension) will be 'seconded' if the ext element isn't provided. Change to
return undefined
if this is undesireable.Love the extension by the way, makes things much nicer to look at.