23966 2009-02-15 07:24:29 -0800 REGRESSION: Custom arrow navigation functionality doesn't work at dropular.net 2009-03-18 13:08:28 -0700 1 1 1 Unclassified WebKit DOM 528+ (Nightly build) Mac (Intel) OS X 10.5 RESOLVED FIXED http://dropular.net/ InRadar, NeedsReduction, Regression P1 Normal --- 1 pattern86 webkit-unassigned oldest_to_newest 109800 0 pattern86 2009-02-15 07:24:29 -0800 I think this is javascript-related, but I'm not sure. On this website you can navigate posts by using the up/down arrows and you can navigate pages (forward/back) using the left/right arrows (see the top right of the page for explanation). The left/right functionality works, but the up/down does not anymore. I just noticed this in the r40957 build from 2009-02-13. 109828 1 ap 2009-02-15 23:28:16 -0800 Confirmed as a regression with r41016. 109829 2 ap 2009-02-15 23:29:19 -0800 <rdar://problem/6589657> 113567 3 ap 2009-03-13 02:25:05 -0700 The problem here is with a broken getBoundingClientRect implementation, which jQuery decides to use when available. Try e.g. javascript:alert(document.getElementById("drop7").getBoundingClientRect().top) 114215 4 28728 sam 2009-03-18 12:49:25 -0700 Created attachment 28728 patch 114216 5 sam 2009-03-18 13:08:28 -0700 Fixed in r41811. 28728 2009-03-18 12:49:25 -0700 2009-03-18 12:52:13 -0700 patch emptyInline.diff text/plain 4561 sam SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiA0MTgxMCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTkgQEAKKzIwMDktMDMtMTggIFNhbSBXZWluaWcgIDxzYW1Ad2Via2l0Lm9yZz4K KworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBGaXggZm9y IGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMzk2NgorICAgICAgICBS RUdSRVNTSU9OOiBDdXN0b20gYXJyb3cgbmF2aWdhdGlvbiBmdW5jdGlvbmFsaXR5IGRvZXNuJ3Qg d29yayBhdCBkcm9wdWxhci5uZXQKKyAgICAgICAgPHJkYXI6Ly9wcm9ibGVtLzY1ODk2NTc+CisK KyAgICAgICAgTWF0Y2ggdGhlIENTU09NIHNwZWMgZm9yIGdldENsaWVudFJlY3RzIGFuZCBnZXRC b3VuZGluZ0NsaWVudFJlY3QgYnkKKyAgICAgICAgcmV0dXJuaW5nIGEgMHgwIHJlY3QgYXQgdGhl IGNvcnJlY3QgdG9wL2xlZnQgcG9zaXRpb24gZm9yIGVtcHR5IGlubGluZQorICAgICAgICBib3hl cy4KKworICAgICAgICAqIHJlbmRlcmluZy9SZW5kZXJJbmxpbmUuY3BwOgorICAgICAgICAoV2Vi Q29yZTo6UmVuZGVySW5saW5lOjphYnNvbHV0ZVJlY3RzKToKKyAgICAgICAgKFdlYkNvcmU6OlJl bmRlcklubGluZTo6YWJzb2x1dGVRdWFkcyk6CisKIDIwMDktMDMtMTggIERhdmlkIExldmluICA8 bGV2aW5AY2hyb21pdW0ub3JnPgogCiAgICAgICAgIFJldmlld2VkIGJ5IEVyaWMgU2VpZGVsLgpJ bmRleDogV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVySW5saW5lLmNwcAo9PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBX ZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJJbmxpbmUuY3BwCShyZXZpc2lvbiA0MTc0MCkKKysrIFdl YkNvcmUvcmVuZGVyaW5nL1JlbmRlcklubGluZS5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTQyMiw4 ICs0MjIsMTEgQEAgdm9pZCBSZW5kZXJJbmxpbmU6OnBhaW50KFBhaW50SW5mbyYgcGFpbgogCiB2 b2lkIFJlbmRlcklubGluZTo6YWJzb2x1dGVSZWN0cyhWZWN0b3I8SW50UmVjdD4mIHJlY3RzLCBp bnQgdHgsIGludCB0eSwgYm9vbCB0b3BMZXZlbCkKIHsKLSAgICBmb3IgKElubGluZVJ1bkJveCog Y3VyciA9IGZpcnN0TGluZUJveCgpOyBjdXJyOyBjdXJyID0gY3Vyci0+bmV4dExpbmVCb3goKSkK LSAgICAgICAgcmVjdHMuYXBwZW5kKEludFJlY3QodHggKyBjdXJyLT54KCksIHR5ICsgY3Vyci0+ eSgpLCBjdXJyLT53aWR0aCgpLCBjdXJyLT5oZWlnaHQoKSkpOworICAgIGlmIChJbmxpbmVSdW5C b3gqIGN1cnIgPSBmaXJzdExpbmVCb3goKSkgeworICAgICAgICBmb3IgKDsgY3VycjsgY3VyciA9 IGN1cnItPm5leHRMaW5lQm94KCkpCisgICAgICAgICAgICByZWN0cy5hcHBlbmQoSW50UmVjdCh0 eCArIGN1cnItPngoKSwgdHkgKyBjdXJyLT55KCksIGN1cnItPndpZHRoKCksIGN1cnItPmhlaWdo dCgpKSk7CisgICAgfSBlbHNlCisgICAgICAgIHJlY3RzLmFwcGVuZChJbnRSZWN0KHR4LCB0eSwg MCwgMCkpOwogCiAgICAgaWYgKGNvbnRpbnVhdGlvbigpICYmIHRvcExldmVsKSB7CiAgICAgICAg IGlmIChjb250aW51YXRpb24oKS0+aXNCb3goKSkgewpAQCAtNDM5LDExICs0NDIsMTQgQEAgdm9p ZCBSZW5kZXJJbmxpbmU6OmFic29sdXRlUmVjdHMoVmVjdG9yPAogCiB2b2lkIFJlbmRlcklubGlu ZTo6YWJzb2x1dGVRdWFkcyhWZWN0b3I8RmxvYXRRdWFkPiYgcXVhZHMsIGJvb2wgdG9wTGV2ZWwp CiB7Ci0gICAgZm9yIChJbmxpbmVSdW5Cb3gqIGN1cnIgPSBmaXJzdExpbmVCb3goKTsgY3Vycjsg Y3VyciA9IGN1cnItPm5leHRMaW5lQm94KCkpIHsKLSAgICAgICAgRmxvYXRSZWN0IGxvY2FsUmVj dChjdXJyLT54KCksIGN1cnItPnkoKSwgY3Vyci0+d2lkdGgoKSwgY3Vyci0+aGVpZ2h0KCkpOwot ICAgICAgICBxdWFkcy5hcHBlbmQobG9jYWxUb0Fic29sdXRlUXVhZChsb2NhbFJlY3QpKTsKLSAg ICB9Ci0gICAgCisgICAgaWYgKElubGluZVJ1bkJveCogY3VyciA9IGZpcnN0TGluZUJveCgpKSB7 CisgICAgICAgIGZvciAoOyBjdXJyOyBjdXJyID0gY3Vyci0+bmV4dExpbmVCb3goKSkgeworICAg ICAgICAgICAgRmxvYXRSZWN0IGxvY2FsUmVjdChjdXJyLT54KCksIGN1cnItPnkoKSwgY3Vyci0+ d2lkdGgoKSwgY3Vyci0+aGVpZ2h0KCkpOworICAgICAgICAgICAgcXVhZHMuYXBwZW5kKGxvY2Fs VG9BYnNvbHV0ZVF1YWQobG9jYWxSZWN0KSk7CisgICAgICAgIH0KKyAgICB9IGVsc2UKKyAgICAg ICAgcXVhZHMuYXBwZW5kKGxvY2FsVG9BYnNvbHV0ZVF1YWQoRmxvYXRSZWN0KCkpKTsKKwogICAg IGlmIChjb250aW51YXRpb24oKSAmJiB0b3BMZXZlbCkKICAgICAgICAgY29udGludWF0aW9uKCkt PmFic29sdXRlUXVhZHMocXVhZHMsIHRvcExldmVsKTsKIH0KSW5kZXg6IExheW91dFRlc3RzL0No YW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cJKHJldmlzaW9uIDQx ODEwKQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEs MTQgQEAKKzIwMDktMDMtMTggIFNhbSBXZWluaWcgIDxzYW1Ad2Via2l0Lm9yZz4KKworICAgICAg ICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBVcGRhdGUgdGVzdCByZXN1 bHRzIGZvciBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjM5NjYKKyAg ICAgICAgUkVHUkVTU0lPTjogQ3VzdG9tIGFycm93IG5hdmlnYXRpb24gZnVuY3Rpb25hbGl0eSBk b2Vzbid0IHdvcmsgYXQgZHJvcHVsYXIubmV0CisgICAgICAgIDxyZGFyOi8vcHJvYmxlbS82NTg5 NjU3PgorCisgICAgICAgICogZmFzdC9kb20vZ2V0Qm91bmRpbmdDbGllbnRSZWN0LWV4cGVjdGVk LnR4dDoKKyAgICAgICAgKiBmYXN0L2RvbS9nZXRDbGllbnRSZWN0cy1leHBlY3RlZC50eHQ6CisK IDIwMDktMDMtMTggIEFsZXhleSBQcm9za3VyeWFrb3YgIDxhcEB3ZWJraXQub3JnPgogCiAgICAg ICAgIFJldmlld2VkIGJ5IERhcmluIEFkbGVyLgpJbmRleDogTGF5b3V0VGVzdHMvZmFzdC9kb20v Z2V0Qm91bmRpbmdDbGllbnRSZWN0LWV4cGVjdGVkLnR4dAo9PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRU ZXN0cy9mYXN0L2RvbS9nZXRCb3VuZGluZ0NsaWVudFJlY3QtZXhwZWN0ZWQudHh0CShyZXZpc2lv biA0MTc0MCkKKysrIExheW91dFRlc3RzL2Zhc3QvZG9tL2dldEJvdW5kaW5nQ2xpZW50UmVjdC1l eHBlY3RlZC50eHQJKHdvcmtpbmcgY29weSkKQEAgLTE5NywxNSArMTk3LDE1IEBAIFBBU1MgdGVz dFJlY3QuYm90dG9tIGlzIHRlc3RSZWN0LnRvcCArIHQKIFBBU1MgdGVzdFJlY3QucmlnaHQgaXMg dGVzdFJlY3QubGVmdCArIHRlc3RSZWN0LndpZHRoCiAKIENsaWVudCBib3VuZGluZyByZWN0IGZv ciAjMTUKLXRlc3RSZWN0LnRvcCAwCi10ZXN0UmVjdC5sZWZ0IDAKK3Rlc3RSZWN0LnRvcCAyNDUy Cit0ZXN0UmVjdC5sZWZ0IDgKIHRlc3RSZWN0LndpZHRoIDAKIHRlc3RSZWN0LmhlaWdodCAwCi10 ZXN0UmVjdC5ib3R0b20gMAotdGVzdFJlY3QucmlnaHQgMAordGVzdFJlY3QuYm90dG9tIDI0NTIK K3Rlc3RSZWN0LnJpZ2h0IDgKIAotRkFJTCB0ZXN0UmVjdC50b3Agc2hvdWxkIGJlIDI0NTIuIFdh cyAwLgotRkFJTCB0ZXN0UmVjdC5sZWZ0IHNob3VsZCBiZSA4LiBXYXMgMC4KK1BBU1MgdGVzdFJl Y3QudG9wIGlzIDI0NTIKK1BBU1MgdGVzdFJlY3QubGVmdCBpcyA4CiBQQVNTIHRlc3RSZWN0Lndp ZHRoIGlzIDAKIFBBU1MgdGVzdFJlY3QuaGVpZ2h0IGlzIDAKIFBBU1MgdGVzdFJlY3QuYm90dG9t IGlzIHRlc3RSZWN0LnRvcCArIHRlc3RSZWN0LmhlaWdodApJbmRleDogTGF5b3V0VGVzdHMvZmFz dC9kb20vZ2V0Q2xpZW50UmVjdHMtZXhwZWN0ZWQudHh0Cj09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRl c3RzL2Zhc3QvZG9tL2dldENsaWVudFJlY3RzLWV4cGVjdGVkLnR4dAkocmV2aXNpb24gNDE3NDAp CisrKyBMYXlvdXRUZXN0cy9mYXN0L2RvbS9nZXRDbGllbnRSZWN0cy1leHBlY3RlZC50eHQJKHdv cmtpbmcgY29weSkKQEAgLTQxLDcgKzQxLDcgQEAgQ2xpZW50IGJvdW5kaW5nIHJlY3RzIGZvciAj MTQKIFBBU1MgdGVzdFJlY3RzLmxlbmd0aCBpcyAzCiAKIENsaWVudCBib3VuZGluZyByZWN0cyBm b3IgIzE1Ci1GQUlMIHRlc3RSZWN0cy5sZW5ndGggc2hvdWxkIGJlIDEuIFdhcyAwLgorUEFTUyB0 ZXN0UmVjdHMubGVuZ3RoIGlzIDEKIAogUEFTUyBzdWNjZXNzZnVsbHlQYXJzZWQgaXMgdHJ1ZQog Cg==