Skip to main content

Bug Tracker

Side navigation

#10782 closed bug (fixed)

Opened November 14, 2011 11:39AM UTC

Closed December 12, 2011 03:44PM UTC

Last modified June 08, 2012 12:38PM UTC

Incorrect calculating width

Reported by: anonymous Owned by: mikesherov
Priority: low Milestone: 1.7.2
Component: css Version: 1.7
Keywords: Cc:
Blocked by: Blocking:
Description

If parent is hidden width() represents width in pixels however it's in percent.

http://jsfiddle.net/CaJNB/

Attachments (0)
Change History (12)

Changed November 14, 2011 07:35PM UTC by timmywil comment:1

component: unfiledcss
milestone: None1.next
priority: undecidedlow
status: newopen

Confirmed. http://jsfiddle.net/timmywil/CaJNB/3/

We run into problems when padding and margin do not have the same unit, so perhaps we should normalize to 0 when the unit is not pixels.

Changed November 14, 2011 07:36PM UTC by timmywil comment:2

owner: → timmywil
status: openassigned

Changed November 19, 2011 02:32AM UTC by mikesherov comment:3

@timmywil, #9945 is the same thing essentially, can we tackle both in 1.7.2 please? I'll gladly jump on it.

Changed November 19, 2011 02:33AM UTC by mikesherov comment:4

@timmywil, #10117 too.

Changed November 19, 2011 06:05PM UTC by timmywil comment:5

@mikesherov: feel free to take them on!

Changed November 26, 2011 02:43AM UTC by dmethvin comment:6

#10894 is a duplicate of this ticket.

Changed December 08, 2011 01:05PM UTC by mikesherov comment:7

owner: timmywilmikesherov

Changed December 12, 2011 03:44PM UTC by timmywil comment:8

resolution: → fixed
status: assignedclosed

When the width/height computed unit is not pixels, return that instead. Fixes #10782.

  • Reordered some of css.js in preparation for jshint undef.

Changeset: 9bea2167c434d5b97643f53da16b04927dc02a7d

Changed December 12, 2011 04:26PM UTC by timmywil comment:9

milestone: 1.next1.7.2

Changed December 16, 2011 04:35PM UTC by dmethvin comment:10

#11041 is a duplicate of this ticket.

Changed December 17, 2011 04:59PM UTC by dmethvin comment:11

#11051 is a duplicate of this ticket.

Changed June 08, 2012 12:38PM UTC by anonymous comment:12

This is still broken. Both the jsfiddle above (using 1.7.2) and production code still show this problem.