search

reinh / jquery-autocomplete

jQuery Autocomplete plugin
http://reinh.github.com/jquery-autocomplete
97 stars 20 forks source link

adjustWidth property #2

Open farinspace opened 13 years ago

farinspace commented 13 years ago

code comments state that this property can have a pixel value set for it, however it looks like the code does not support this. I am not quite sure of the intent of this property... if it were a boolean only then setting it to FALSE would allow the end user to make width adjustments via the css, however if the intent of the property is to allow a pixel value to be set, then additional code is needed to support this.

farinspace commented 13 years ago

in practice its a little cumbersome having to change values in two locations to adjust the width (in js adjustWidth and in the css) ... i wonder if width can be auto detected if it is set in the css already ... the end user would never have to worry about adjustWidth prop

reinh commented 13 years ago

Does seem unnecessarily unDRY.