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.
Attachments (0)
Change History (12)
Changed November 14, 2011 07:35PM UTC by comment:1
component: | unfiled → css |
---|---|
milestone: | None → 1.next |
priority: | undecided → low |
status: | new → open |
Changed November 14, 2011 07:36PM UTC by comment:2
owner: | → timmywil |
---|---|
status: | open → assigned |
Changed November 19, 2011 02:32AM UTC by 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 06:05PM UTC by comment:5
@mikesherov: feel free to take them on!
Changed December 08, 2011 01:05PM UTC by comment:7
owner: | timmywil → mikesherov |
---|
Changed December 12, 2011 03:44PM UTC by comment:8
resolution: | → fixed |
---|---|
status: | assigned → closed |
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 comment:9
milestone: | 1.next → 1.7.2 |
---|
Changed December 16, 2011 04:35PM UTC by comment:10
#11041 is a duplicate of this ticket.
Changed December 17, 2011 04:59PM UTC by comment:11
#11051 is a duplicate of this ticket.
Changed June 08, 2012 12:38PM UTC by comment:12
This is still broken. Both the jsfiddle above (using 1.7.2) and production code still show this problem.
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.