Image link update - Githubissues

doomeer / factorio

Factorio Planner

MIT License

136 stars 39 forks source link

Image link update #27

Closed NixonK closed 7 years ago

NixonK commented 7 years ago

Fixes #25

NixonK commented 7 years ago

Changed to WIP. Fixing the variable names to shorten the special_icons list

NixonK commented 7 years ago

Removed from WIP.

clean out usage of special_icons and special_hrefs; variable names now match in-game item names - which are also now the standard used by the wiki

mpsijm commented 7 years ago

Thanks @NixonK for this pull request :tada: I hope this can be merged soon, then I can normally use the planner again :smile:

doomeer commented 7 years ago

Very nice patch, thanks a lot!

doomeer commented 7 years ago

I changed Char.lowercase_ascii to Char.lowercase. lowercase_ascii is better because lowercase is deprecated, but on the other hand, lowercase_ascii is only available in OCaml 4.03.