Bug Tracker

Opened 12 years ago

Closed 12 years ago

#9300 closed bug (fixed)

outerWidth(true) issue

Reported by: jon301@… Owned by: Timmy Willison
Priority: high Milestone: 1.6.2
Component: dimensions Version: 1.6.1
Keywords: Cc:
Blocked by: Blocking: #9417

Description

Hello,

outerWidth(true) on a hidden element seems to return the margin of the element (without its width)

For example, if my element is 100px width and has a 10px margin-left, outerWidth(true) will return 10. outerWidth() returns 100.

Look at this test case : ​http://jsfiddle.net/CSwNr/

Change History (10)

comment:1 Changed 12 years ago by Timmy Willison

Component: unfiled → dimensions
Milestone: 1.next → 1.6.2
Priority: undecided → blocker
Status: new → open

Well that's not good.

comment:2 Changed 12 years ago by Timmy Willison

The confusing part for me is how outerWidth() returns 100 the first time. That shouldn't work either.

comment:3 Changed 12 years ago by Timmy Willison

Owner: set to Timmy Willison
Status: open → assigned

Ok, this makes sense. The fallback is not getting applied because val is not 0 after the getWH call due to the margin addition. Will do a fix.

comment:5 Changed 12 years ago by Timmy Willison

Blocking: 9417 added

(In #9417) ​http://jsfiddle.net/timmywil/S8H2w/1/

It would be great if we could get this working, but it's not easy.

Related to #9300. This is different in that this element does not have a width set in its styles to retrieve as a fallback.

comment:6 Changed 12 years ago by Timmy Willison

Priority: blocker → low

comment:7 Changed 12 years ago by Timmy Willison

Priority: low → high

comment:10 Changed 12 years ago by Mike Sherov

Resolution: → fixed
Status: assigned → closed

Add margin after checking width. Add tests. Fixes #9441. Fixes #9300.

Changeset: 80ad14bd14467c547c2867f2677ca581aa29bf33

Note: See TracTickets for help on using tickets.