Fix number of arguments on `container.addSlider`

Issue #42 new
Remco Janssen created an issue

Not sure why this is a problem. Should this be converted to an options object?

Warning: 297:13 warning Method 'addSlider' has too many parameters (5). Maximum allowed is 4 max-params


Comments (1)

  1. Rune Gangsø

    Yeah, option object would be better, and it matches the api of the other addX methods.

    Max params is set to 4 because it is difficult to know/nicely code all the params correctly when there are too many. I think after 4 you start struggling with line length and argument order.

  2. Log in to comment