NotificationsYou must be signed in to change notification settings
Fork20.6k
Star59.5k

Commit865469f

authored

CSS: Remove the opacity CSS hook

The consequence is `.css( "opacity" )` will now return an empty string fordetached elements in standard-compliant browsers and "1" in IE & the legacyEdge. That behavior is shared by most other CSS properties which we're notnormalizing either.Closesgh-4593

1 parent89a18de commit865469fCopy full SHA for 865469f

File tree

1 file changed

-12

lines changed

src
- css.js

1 file changed

-12

lines changed

`‎src/css.js`

Lines changed: 1 addition & 12 deletions

Original file line number	Diff line number	Diff line change
`@@ -185,18 +185,7 @@ jQuery.extend( {`
`185`	`185`
`186`	`186`	`// Add in style property hooks for overriding the default`
`187`	`187`	`// behavior of getting and setting a style property`
`188`		`-cssHooks:{`
`189`		`-opacity:{`
`190`		`-get:function(elem,computed){`
`191`		`-if(computed){`
`192`		`-`
`193`		`-// We should always get a number back from opacity`
`194`		`-varret=curCSS(elem,"opacity");`
`195`		`-returnret==="" ?"1" :ret;`
`196`		`-}`
`197`		`-}`
`198`		`-}`
`199`		`-},`
	`188`	`+cssHooks:{},`
`200`	`189`
`201`	`190`	`// Get and set the style property on a DOM Node`
`202`	`191`	`style:function(elem,name,value,extra){`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit865469f

File tree

1 file changed

1 file changed

`‎src/css.js`

0 commit comments