Draggable: Draggable elements receive an explicit height in Firefox only

[orfc]

Firefox is v34.0.5

Made a draggable composed of two parts. Header (with the handle) and content. A button in the header should hide/show the content.

​http://jsfiddle.net/cdv5bLLh/ jquery ui v.1.10.3, works ok in firefox,

​http://jsfiddle.net/ck93q4of/53/ jquery ui v.1.11.2, works before dragging in firefox, but dragging then gives height css property a set value, so hiding/showing alters the display of the content but the containing box won't change in size (as the height property isn't auto anymore)

Looking through the new code it seems a call to normalizeRightBottom sets the height to a calculated value if the right and bottom position values aren't detected as "auto". Unfortunately firefox never returns values as "auto"

​https://developer.mozilla.org/en/docs/Web/API/window.getComputedStyle#Notes

In Firefox, properties with the value auto return the used value, not the value auto

[orfc]

Whoops, make the version that works in firefox as ​http://jsfiddle.net/cdv5bLLh/2/

[tj.vantoll]

I would agree that we shouldn't set height unless we need to. I verified that the this.helper.css( "bottom" ) call in _normalizeRightBottom() returns "auto" in Chrome but a pixel amount (e.g. "-142px") in Firefox.

This changed in 1.11.2 per this commit: ​https://github.com/jquery/jquery-ui/commit/8eca7b8f45885d20c13f1bf64cad8bee5fc1d5c5.

[scottgonzalez]

#10729 is a duplicate of this ticket.

[mikesherov]

Firefox is following the spec here in that it uses the "resolved value" which for trbl is always a pixel value. There is currently no way in the CSSOM to know simultaneously whether an element has auto applied or not. We need to accept one of two answers: 1. Auto height is broken. or 2. Right/bottom is broken.

I'll go to the CSS working group with this in the new year, but don't expect any miracles ;)

Scott, what do you want to do here?

[scottgonzalez]

I'd say we revert 8eca7b8f45885d20c13f1bf64cad8bee5fc1d5c5, close #7772 as a duplicate of #3011, and continue to leave #3011 open for the possibility of eventually adding an option to make this work.

[tj.vantoll]

Replying to scott.gonzalez:

I'd say we revert 8eca7b8f45885d20c13f1bf64cad8bee5fc1d5c5, close #7772 as a duplicate of #3011, and continue to leave #3011 open for the possibility of eventually adding an option to make this work.

+1

[mikesherov]

Ok. I'll do that then.

[tj.vantoll]

#11776 is a duplicate of this ticket.

[tj.vantoll]

Test case from #11776: ​http://jsfiddle.net/oo1he4px/

[tj.vantoll]

PR: ​https://github.com/jquery/jquery-ui/pull/1531

[TJ VanToll]

Revert "Draggable: Set explicit width/height instead of right/bottom"

This reverts commit 8eca7b8f45885d20c13f1bf64cad8bee5fc1d5c5.

Fixes #10725

Changeset: d0ea32e3ad613eaaa523d0c88c776dab2b26b25f

[scottgonzalez]

#12232 is a duplicate of this ticket.