Solution needed for selectlist [data-resize]=auto when display: none- fixes #1069

[BenjaminNeilDavis]

• fixes #1069
• Solution needed for selectlist [data-resize]=auto when display: none
• added catch if width returns as 0 and displays selectlist as normal.
• different test examples http://jsbin.com/beyogiqizi/

[swilliamset]

uncomfortable with an !important here. is there a place we can do the following

• check if visible
• if hidden
  • change visibility transparent
  • show
  • measure
  • reset

[BenjaminNeilDavis]

This is just for the dummy selectlist I create. The element I append to the body to get the width of the dropdown items then remove once I have a width.

[swilliamset]

lets provide a more explicit class name then to make it clear it's a dummy
.dummy-selectlist

[cmcculloh-kr]

If "sizer" is good enough for the js variable name, why are we deviating from that in the CSS? Now it is inconsistent and even more confusing.

[swilliamset]

Sizer is fine. We can move back. I wasn't looking at the context of the css class.

[cmcculloh-kr]

Are we able to test this in a unit test?

[cmcculloh-kr]

Your "hidden on load" test example is breaking:

[BenjaminNeilDavis]

@CormacMcCarthy Were you testing it in the selectlist test markup?

[cmcculloh-kr]

@BenjaminNeilDavis I clicked on the link you provided:

(http://jsbin.com/beyogiqizi/)

and it was borked.

[BenjaminNeilDavis]

@CormacMcCarthy I was just mainly providing the html for the tests (because dev file is now ignored). The js / css for fuelux is pointing to the CDN which doesnt have my changes yet. If you were to copy the html / js from the example and use it on the dev.html page you would be able to see the change.

[cmcculloh-kr]

Indeed! Thanks :)

[cmcculloh-kr]

I think a bad squash resulted in losing some code.