search

twolfson / spritesmith

Utility that takes sprites and converts them into a stylesheet and its coordinates
MIT License
914 stars 56 forks source link

Update README.md #50

Closed tmcw closed 9 years ago

tmcw commented 9 years ago

Add default padding value

twolfson commented 9 years ago

Can you elaborate on why this change is necessary? I believe it is straightforward that if the option is not present, then there is no padding.

tmcw commented 9 years ago

I think that docs should err on the side of being explicit rather than relying on assumptions like that - I certainly didn't assume that the default padding was zero, and before changing a value in my application that uses spritesmith, found it necessary to read the code to confirm the default value.

twolfson commented 9 years ago

Being overly explicit can waste unnecessary time maintaining documentation. I am going to decide it is fine for now. If more people complain about this, then I will reconsider adding it to the documentation.