remove placeholder if it's not going to be used

[cabbey]

building with --no-force-geojson currently leaves %JSFILE% in the generated index.html, which then renders in the browser and kills an entire text line worth of height on the display.

(note not sure this is the best solution, I just aped the code that was in the if leg to replace it with nothing and it works.)

[cabbey]

An alternate approach might have been to make the placeholder a valid html comment and replaced the whole thing, something like <!-- %JSFILE% -->, this way in the case where it's not replaced, it isn't renderable content. (I didn't think of that at 2am when I was trying to make a map to show my son today. ;)

[tomnolan]

An alternate approach might have been to make the placeholder a valid html comment and replaced the whole thing, something like <!-- %JSFILE% -->, this way in the case where it's not replaced, it isn't renderable content. (I didn't think of that at 2am when I was trying to make a map to show my son today. ;)

You're right on with this @cabbey, but as my wife says, it's six in one and a half dozen in the other. Thanks for contributing.

PR looks good to me!

[tomnolan]

Resolves #49

[cabbey]

Ah, as a project you require an issue for all PRs? I'll keep that in mind for the future.