Side navigation
#9233 closed bug (fixed)
Opened May 11, 2011 10:27AM UTC
Closed October 28, 2011 02:55PM UTC
getPixelVaue method in jquery
| Reported by: | zhengzongyi@gmail.com | Owned by: | timmywil |
|---|---|---|---|
| Priority: | low | Milestone: | 1.next |
| Component: | css | Version: | 1.6 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Attachments (0)
Change History (7)
Changed May 11, 2011 12:10PM UTC by comment:1
| component: | unfiled → misc |
|---|---|
| owner: | → zhengzongyi@gmail.com |
| status: | new → pending |
Changed May 12, 2011 02:05AM UTC by comment:2
| status: | pending → new |
|---|
please run it in IE(my browser is version 8).
Changed May 12, 2011 02:54PM UTC by comment:3
| component: | misc → css |
|---|---|
| owner: | zhengzongyi@gmail.com → timmywil |
| priority: | undecided → low |
| status: | new → assigned |
I can't think of a use case where that would cause something to break, but I'll take a look as to why it's changing the runtime value.
Changed October 23, 2011 02:50PM UTC by comment:4
Looks like the change here had some copypasta: https://github.com/jquery/jquery/commit/8099cdce800d45109b961e3521fc0080e9a876c6
I'll open a PR just in case we care to fix this.
Changed October 23, 2011 03:56PM UTC by comment:5
Changed October 23, 2011 05:12PM UTC by comment:6
Nope, no copypasta there, everything I did was intentional. If you'd like to discuss my changes line-by-line, I'd be glad to. The problem was runtimeStyle throwing exceptions in Opera, but clearly we had no tests for cases like this. (see: #7608 )
TBH, that fiddle feels contrived...
Thanks for taking the time to contribute to the jQuery project! Please provide a reduced test case on http://jsFiddle.net that reproduces the issue experienced to help us assess your ticket!
Additionally, test against the
jQuery (edge)version to ensure the issue still exists.