Make outerWidth/Height a setter

[jzaefferer]

The math required by the regular boxmodel leaves it traces in a lot of places where we need to set the width or height of an element that has border, padding or margin.

jQuery UI currently extends inner/outer/Width/Height as setters, but we only use the outerXxx methods: ​https://github.com/jquery/jquery-ui/blob/master/ui/jquery.ui.core.js#L132-L174

By actually integrating that in core, the require code would obviously be a lot less. For consistency it might make sense to make both outerXxx and innerXxx a setter, but really only outer is important.

[dmethvin]

#10994 is a duplicate of this ticket.

[jzaefferer]

+1

[jaubourg]

+0, Not sure here, what the cost in size?

[dmethvin]

+1, Yes, but as with everything in 1.8 contingent on final size. This intersects with our box-sizing: border-box issues as well.

[mikesherov]

+1, This is something people would expect jQuery to handle, and I'd love to work on this, so yes. Also, we can pretty much just rip the code straight from jquery ui. I do have some concerns though with code paths for hidden elements, but should be fun to implement.

[Timmy Willison]

-1

[Rick Waldron]

+1

[ALLPRO]

+1 -- have wondered why this was not done a long time ago

[mikesherov]

​https://github.com/jquery/jquery/pull/783

[Timmy Willison]

I'm still a -1 on this feauture, especially given the added size to core.

[mikesherov]

@timmywil, I hear the concern. There are strategies for smaller size that I outlined in the PR. Thoughts?

[Mike Sherov]

Fix #10877. Make outerWidth/Height a setter. Closes gh-783.

Changeset: e0151e5827d7091f311c82d9f951aaaa2688ba8c

[jlgrall]

Why is this not documented in ​http://api.jquery.com/outerWidth/ and ​http://api.jquery.com/outerHeight/ ?

[jzaefferer]

API docs ticket is still open: ​https://github.com/jquery/api.jquery.com/issues/98