philc/vimium — pull request #688 by smblott-github (2012-10-28 09:38:37+00:00)
Title: World's smallest bug: Correct handling of default value for scrollStepSize

`scrollStepSize` is the only setting whose default value is numeric.

The following test:

```
# don't store the value if it is equal to the default, so we can change the defaults in the future
if (value == @defaults[key])
@clear(key)
```

from `settings.coffee` *never* succeeds for scrollStepSize:
- because `==` compiles to `===` and the types don't match.

Therefore, the default value of `scrollStepSize` is *always* stored in `localStorage` ... which obviates the intention of the test quoted above.

***

This bug is so small that I thought it might be ok to skip raising it as an "Issue". My apologies if that's not ok.