Side navigation
#8401 closed enhancement (fixed)
Opened February 28, 2011 09:16AM UTC
Closed April 12, 2011 04:36AM UTC
minor enhancement to opacity cssHooks
| Reported by: | lrbabe | Owned by: | jitter |
|---|---|---|---|
| Priority: | low | Milestone: | 1.6 |
| Component: | css | Version: | 1.5.1 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
Why are we doing
var filter = style.filter || ""; style.filter = style.filter + ' ' + opacity;
When we could do
var filter = style.filter || ""; // reuse the 'filter' var style.filter = filter + ' ' + opacity;
Improve jQuery, one DOM access at a time.
The actual code is slightly longer, but the modification is equally tiny, see following push request.
Attachments (0)
Change History (3)
Changed February 28, 2011 09:19AM UTC by comment:1
Changed February 28, 2011 10:45AM UTC by comment:2
| component: | unfiled → css |
|---|---|
| owner: | → jitter |
| priority: | undecided → low |
| status: | new → assigned |
Changed April 12, 2011 04:36AM UTC by comment:3
| milestone: | 1.next → 1.6 |
|---|---|
| resolution: | → fixed |
| status: | assigned → closed |
Landed.
Here is the pull request: https://github.com/jquery/jquery/pull/254/files