18076 2008-03-25 14:17:38 -0700 Box disappears after CSS Transition completes 2008-03-31 15:31:34 -0700 1 1 1 Unclassified WebKit CSS 528+ (Nightly build) Mac OS X 10.5 RESOLVED FIXED P2 Normal --- 1 farukates webkit-unassigned mitz oldest_to_newest 75152 0 farukates 2008-03-25 14:17:38 -0700 I'm transitioning a test box's border-radius (sidebar on that below) on :hover and after the transition is done the background disappears; if I hover out the entire box becomes invisible (flow doesn't change) including text content. Test case attached. Sidebar: to get the border radius property to transition, you actually have to use -webkit-transition-property: border-radius; (instead of -webkit-border-radius) which is inconsistent with how we currently have to make border radius work in the first place. Bug exists in both Safari 3.1 and latest Webkit nightly. 75153 1 20041 farukates 2008-03-25 14:18:37 -0700 Created attachment 20041 Sample of :hover state transition and box disappearing 75158 2 farukates 2008-03-25 14:26:59 -0700 Additional bug (should I file a separate report?): the background-color and text color transition as well, even though the test case only specifies -webkit-transition-property: border-radius; (shouldn't that make all other properties NOT transition, but simply instantly switch?) 75257 3 mitz 2008-03-25 19:58:08 -0700 (In reply to comment #2) > Additional bug (should I file a separate report?): the background-color and > text color transition as well, even though the test case only specifies > -webkit-transition-property: border-radius; (In reply to comment #0) > Sidebar: to get the border radius property to transition, you actually have to > use -webkit-transition-property: border-radius; (instead of > -webkit-border-radius) which is inconsistent with how we currently have to make > border radius work in the first place. Actually any invalid property keyword you will put there will result in the behavior you see with "border-radius" (which is an invalid property keyword). That the shorthand property "-webkit-border-radius" is not supported is a different issue. > Additional bug (should I file a separate report?): the background-color and > text color transition as well, even though the test case only specifies > -webkit-transition-property: border-radius; That is all part of this bug: specifying an invalid property keyword caused all properties to animate and the box's background to vanish. 75258 4 mitz 2008-03-25 20:03:05 -0700 Specifically, it appears that animating 'color' and 'background-color' simultaneously is what leads to the background color being reset to transparent in the end of the transition. 75299 5 farukates 2008-03-26 08:00:29 -0700 I see — so the bugs here are actually two entirely different bugs than I thought: 1) transitioning both background-color and color simultaneously makes them both end up transparent at the end of the transition; 2) the shorthand for -webkit-border-radius: isn't implemented (but it is for just border-radius, which it shouldn't be yet, right?) I got it to work as intended by being fully explicit (-webkit-border-top-right-radius & …bottom-right…) so that behavior is indeed fully operational. 75890 6 20228 mitz 2008-03-30 18:50:08 -0700 Created attachment 20228 Fix the StyleBackgroundData copy constructor; make BLEND_MAYBE_INVALID_COLOR more efficient 76007 7 20228 hyatt 2008-03-31 15:23:40 -0700 Comment on attachment 20228 Fix the StyleBackgroundData copy constructor; make BLEND_MAYBE_INVALID_COLOR more efficient r=me 76012 8 mitz 2008-03-31 15:31:34 -0700 Fixed in <http://trac.webkit.org/projects/webkit/changeset/31485>. 20041 2008-03-25 14:18:37 -0700 2008-03-25 14:18:37 -0700 Sample of :hover state transition and box disappearing border-radius.html text/html 878 farukates PCFET0NUWVBFIGh0bWwgUFVCTElDICItLy9XM0MvL0RURCBYSFRNTCAxLjAgU3RyaWN0Ly9FTiIK CSJodHRwOi8vd3d3LnczLm9yZy9UUi94aHRtbDEvRFREL3hodG1sMS1zdHJpY3QuZHRkIj4KPGh0 bWwgeG1sbnM9Imh0dHA6Ly93d3cudzMub3JnLzE5OTkveGh0bWwiIGxhbmc9ImVuIj4KPGhlYWQ+ Cgk8bWV0YSBodHRwLWVxdWl2PSJjb250ZW50LXR5cGUiIGNvbnRlbnQ9InRleHQvaHRtbDsgY2hh cnNldD11dGYtOCIgLz4KCTx0aXRsZT5XZWJraXQgQ1NTIEFuaW1hdGlvbnMgdGVzdDwvdGl0bGU+ Cgo8c3R5bGUgdHlwZT0idGV4dC9jc3MiIG1lZGlhPSJhbGwiPgpib2R5IHsKCXBhZGRpbmc6IDUw cHg7Cn0KI3Rlc3Rib3ggewoJd2lkdGg6IDIwMHB4OwoJaGVpZ2h0OiAxMDBweDsKCWxpbmUtaGVp Z2h0OiAxMDBweDsKCXRleHQtYWxpZ246IGNlbnRlcjsKCWNvbG9yOiAjZmZmOwoJYmFja2dyb3Vu ZDogIzIzNDQ0MzsKCS13ZWJraXQtYm9yZGVyLXJhZGl1czogMTBweDsKCS13ZWJraXQtdHJhbnNp dGlvbi1wcm9wZXJ0eTogYm9yZGVyLXJhZGl1czsKCS13ZWJraXQtdHJhbnNpdGlvbi1kdXJhdGlv bjogMnM7Cgp9CgojdGVzdGJveDpob3ZlciB7Cgljb2xvcjogIzAwMDsKCWJhY2tncm91bmQtY29s b3I6ICNmOGZmODA7Cgktd2Via2l0LWJvcmRlci1yYWRpdXM6IDUwcHg7Cn0KPC9zdHlsZT4KCjwv aGVhZD4KPGJvZHk+Cgo8ZGl2IGlkPSJ0ZXN0Ym94Ij4KCVRoaXMgaXMgdGhlIHRlc3QgYm94Cjwv ZGl2PgoKPHA+VGhpcyBpcyBhIHBhcmFncmFwaCB1bmRlcm5lYXRoIHRoZSBib3ggdG8gc2hvdyBo b3cgdGhlIGJveCBvbmx5IGJlY29tZXMgaW52aXNpYmxlLCBidXQgcmVtYWlucyBpbiB0aGUgZmxv dy48L3A+Cgo8L2JvZHk+CjwvaHRtbD4= 20228 2008-03-30 18:50:08 -0700 2008-03-31 15:23:40 -0700 Fix the StyleBackgroundData copy constructor; make BLEND_MAYBE_INVALID_COLOR more efficient 18076_r1.diff text/plain 2911 mitz SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiAzMTQ0OCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTYgQEAKKzIwMDgtMDMtMzAgIERhbiBCZXJuc3RlaW4gIDxtaXR6QGFwcGxlLmNv bT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAtIGZp eCBodHRwOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xODA3NgorICAgICAgICAg IEJveCBkaXNhcHBlYXJzIGFmdGVyIENTUyBUcmFuc2l0aW9uIGNvbXBsZXRlcworCisgICAgICAg ICogcGFnZS9BbmltYXRpb25Db250cm9sbGVyLmNwcDogQ2hhbmdlZCB0aGUgQkxFTkRfTUFZQkVf SU5WQUxJRF9DT0xPUgorICAgICAgICBtYWNybyB0byBub3QgYmxlbmQgdHdvIGludmFsaWQgY29s b3JzLgorICAgICAgICAqIHJlbmRlcmluZy9SZW5kZXJTdHlsZS5jcHA6CisgICAgICAgIChXZWJD b3JlOjpTdHlsZUJhY2tncm91bmREYXRhOjpTdHlsZUJhY2tncm91bmREYXRhKTogQWRkZWQgbWlz c2luZworICAgICAgICBjb3B5aW5nIG9mIHRoZSBtX2NvbG9yIGRhdGEgbWVtYmVyLgorCiAyMDA4 LTAzLTMwICBEYW4gQmVybnN0ZWluICA8bWl0ekBhcHBsZS5jb20+CiAKICAgICAgICAgUmV2aWV3 ZWQgYnkgU2FtIFdlaW5pZy4KSW5kZXg6IFdlYkNvcmUvcGFnZS9BbmltYXRpb25Db250cm9sbGVy LmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL3BhZ2UvQW5pbWF0aW9uQ29udHJvbGxlci5jcHAJ KHJldmlzaW9uIDMxNDQ2KQorKysgV2ViQ29yZS9wYWdlL0FuaW1hdGlvbkNvbnRyb2xsZXIuY3Bw CSh3b3JraW5nIGNvcHkpCkBAIC0zMzMsMTcgKzMzMywxOSBAQCBzdGF0aWMgaW5saW5lIEVWaXNp YmlsaXR5IGJsZW5kRnVuYyhFVmlzCiAgICAgaWYgKChtX3Byb3BlcnR5ID09IGNBbmltYXRlQWxs ICYmICFhbmltYXRpb24tPmhhc0FuaW1hdGlvbkZvclByb3BlcnR5KHByb3ApKSB8fCBtX3Byb3Bl cnR5ID09IHByb3ApIHsgXAogICAgICAgICBDb2xvciBmcm9tQ29sb3IgPSBtX2Zyb21TdHlsZS0+ Z2V0dGVyKCk7IFwKICAgICAgICAgQ29sb3IgdG9Db2xvciA9IG1fdG9TdHlsZS0+Z2V0dGVyKCk7 IFwKLSAgICAgICAgaWYgKCFmcm9tQ29sb3IuaXNWYWxpZCgpKSBcCi0gICAgICAgICAgICBmcm9t Q29sb3IgPSBtX2Zyb21TdHlsZS0+Y29sb3IoKTsgXAotICAgICAgICBpZiAoIXRvQ29sb3IuaXNW YWxpZCgpKSBcCi0gICAgICAgICAgICB0b0NvbG9yID0gbV90b1N0eWxlLT5jb2xvcigpOyBcCi0g ICAgICAgIGlmIChmcm9tQ29sb3IgIT0gdG9Db2xvcikge1wKLSAgICAgICAgICAgIG1fZmluaXNo ZWQgPSBmYWxzZTsgXAotICAgICAgICAgICAgaWYgKCFhbmltYXRlZFN0eWxlKSBcCi0gICAgICAg ICAgICAgICAgYW5pbWF0ZWRTdHlsZSA9IG5ldyAocmVuZGVyZXItPnJlbmRlckFyZW5hKCkpIFJl bmRlclN0eWxlKCp0YXJnZXRTdHlsZSk7IFwKLSAgICAgICAgICAgIGFuaW1hdGVkU3R5bGUtPnNl dHRlcihibGVuZEZ1bmMoZnJvbUNvbG9yLCB0b0NvbG9yLCBwcm9ncmVzcygpKSk7XAotICAgICAg ICAgICAgaWYgKG1fcHJvcGVydHkgPT0gcHJvcCkgXAotICAgICAgICAgICAgICAgIHJldHVybjsg XAorICAgICAgICBpZiAoZnJvbUNvbG9yLmlzVmFsaWQoKSB8fCB0b0NvbG9yLmlzVmFsaWQoKSkg eyBcCisgICAgICAgICAgICBpZiAoIWZyb21Db2xvci5pc1ZhbGlkKCkpIFwKKyAgICAgICAgICAg ICAgICBmcm9tQ29sb3IgPSBtX2Zyb21TdHlsZS0+Y29sb3IoKTsgXAorICAgICAgICAgICAgaWYg KCF0b0NvbG9yLmlzVmFsaWQoKSkgXAorICAgICAgICAgICAgICAgIHRvQ29sb3IgPSBtX3RvU3R5 bGUtPmNvbG9yKCk7IFwKKyAgICAgICAgICAgIGlmIChmcm9tQ29sb3IgIT0gdG9Db2xvcikge1wK KyAgICAgICAgICAgICAgICBtX2ZpbmlzaGVkID0gZmFsc2U7IFwKKyAgICAgICAgICAgICAgICBp ZiAoIWFuaW1hdGVkU3R5bGUpIFwKKyAgICAgICAgICAgICAgICAgICAgYW5pbWF0ZWRTdHlsZSA9 IG5ldyAocmVuZGVyZXItPnJlbmRlckFyZW5hKCkpIFJlbmRlclN0eWxlKCp0YXJnZXRTdHlsZSk7 IFwKKyAgICAgICAgICAgICAgICBhbmltYXRlZFN0eWxlLT5zZXR0ZXIoYmxlbmRGdW5jKGZyb21D b2xvciwgdG9Db2xvciwgcHJvZ3Jlc3MoKSkpO1wKKyAgICAgICAgICAgICAgICBpZiAobV9wcm9w ZXJ0eSA9PSBwcm9wKSBcCisgICAgICAgICAgICAgICAgICAgIHJldHVybjsgXAorICAgICAgICAg ICAgfVwKICAgICAgICAgfVwKICAgICB9XAogCkluZGV4OiBXZWJDb3JlL3JlbmRlcmluZy9SZW5k ZXJTdHlsZS5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyU3R5bGUu Y3BwCShyZXZpc2lvbiAzMTQ0NikKKysrIFdlYkNvcmUvcmVuZGVyaW5nL1JlbmRlclN0eWxlLmNw cAkod29ya2luZyBjb3B5KQpAQCAtMzI5LDYgKzMyOSw3IEBAIFN0eWxlQmFja2dyb3VuZERhdGE6 OlN0eWxlQmFja2dyb3VuZERhdGEKIFN0eWxlQmFja2dyb3VuZERhdGE6OlN0eWxlQmFja2dyb3Vu ZERhdGEoY29uc3QgU3R5bGVCYWNrZ3JvdW5kRGF0YSYgbykKICAgICA6IFJlZkNvdW50ZWQ8U3R5 bGVCYWNrZ3JvdW5kRGF0YT4oKQogICAgICwgbV9iYWNrZ3JvdW5kKG8ubV9iYWNrZ3JvdW5kKQor ICAgICwgbV9jb2xvcihvLm1fY29sb3IpCiAgICAgLCBtX291dGxpbmUoby5tX291dGxpbmUpCiB7 CiB9Cg==