11985 2006-12-26 12:01:19 -0800 REGRESSION: No day/week scrollbar in Google Calendar 2007-02-12 10:58:32 -0800 1 1 1 Unclassified WebKit Layout and Rendering 420+ Mac OS X 10.4 RESOLVED FIXED GoogleBug, InRadar, NeedsReduction, Regression P1 Normal --- 1 dev+webkit webkit-unassigned bdakin koivisto sullivan oldest_to_newest 39701 0 dev+webkit 2006-12-26 12:01:19 -0800 Load Google Calendar (until bug 11982 is fixed, you'll need the r18159 nightly). There are no scrollbars in the day, week, or next 4 days tabs even though the content scrolls with a trackpad/scrollwheel. After switching to the month or agenda tabs and back, the scrollbars are working properly. Scrollbars are fine in shipping Safari. 31240 1 mrowe 2007-01-28 19:06:37 -0800 <rdar://problem/4960260> 24481 2 13109 koivisto 2007-02-10 12:08:05 -0800 Created attachment 13109 make sure overflow:auto scrollbars are enabled Google Calendar apparently dynamically turns overflow:scroll scrollbars into overflow:auto. If they were disabled at that point they would never be enabled 24432 3 sam 2007-02-10 15:37:35 -0800 Antti, as far as I can tell it looks good, but it needs a test case. 24434 4 13109 mjs 2007-02-10 16:19:50 -0800 Comment on attachment 13109 make sure overflow:auto scrollbars are enabled Fix looks good to me. Just needs a test case for LayoutTests. r- for now, r=me if you add a test case. 24413 5 13112 koivisto 2007-02-10 17:14:47 -0800 Created attachment 13112 test case Interestingly this test case fails also in tiger webkit. Since this bug depends on layout timing Calendar scrollbars probably work there just by accident. 24374 6 sam 2007-02-10 18:05:28 -0800 Antti, the test case looks good, now, if you could just bundle it all together. add a Changelog for the LayoutTest and upload it we will be all set. 24145 7 koivisto 2007-02-11 16:05:30 -0800 Based on comment #4 committed as r19566 with the attached LayoutTest 24069 8 sullivan 2007-02-12 09:38:18 -0800 This appears only partly fixed. With tip of tree, if I visit Google calendar in day or week mode, no vertical scroller appears. But if I click on the "month" tab, then back on the "day" or "week" tab, the vertical scroller then appears, and works correctly. So there's some initial state that's still wrong. 24076 9 koivisto 2007-02-12 10:22:24 -0800 Worksforme. Err, are you sure you actually have r19566 or newer? 24079 10 dev+webkit 2007-02-12 10:45:08 -0800 This also works for me with r19573. 24026 11 sullivan 2007-02-12 10:54:23 -0800 Hmm, I definitely have 19573. I'll double-check this. 24027 12 sullivan 2007-02-12 10:58:32 -0800 Ah, I was building a Release version, but testing on a Debug version that wasn't quite up to date. When I test on a version that actually includes this fix, it works just fine. D'oh! 13109 2007-02-10 12:08:05 -0800 2007-02-10 16:19:50 -0800 make sure overflow:auto scrollbars are enabled 11985.patch text/plain 1422 koivisto SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiAxOTU1NCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTYgQEAKKzIwMDctMDItMTAgIEFudHRpIEtvaXZpc3RvICA8YW50dGlAYXBwbGUu Y29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIC0g Zml4ZWQgUkVHUkVTU0lPTjogTm8gZGF5L3dlZWsgc2Nyb2xsYmFyIGluIEdvb2dsZSBDYWxlbmRh cgorICAgICAgICBodHRwOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMTk4NQor CisgICAgICAgIE1ha2Ugc3VyZSBvdmVyZmxvdzphdXRvIHNjcm9sbGJhcnMgYXJlIGFsd2F5cyBl bmFibGVkLiBJZiB0aGV5IHdlcmUgb3ZlcmZsb3c6c2Nyb2xsCisgICAgICAgIGFuZCBkeW5hbWlj YWxseSBjaGFuZ2VkIHRvIGF1dG8gdGhleSBtaWdodCBzdGlsbCBiZSBkaXNhYmxlZC4KKworICAg ICAgICAqIHJlbmRlcmluZy9SZW5kZXJMYXllci5jcHA6CisgICAgICAgIChXZWJDb3JlOjpSZW5k ZXJMYXllcjo6dXBkYXRlU2Nyb2xsSW5mb0FmdGVyTGF5b3V0KToKKwogMjAwNy0wMi0wOSAgQnJh ZHkgRWlkc29uIDxiZWlkc29uQGFwcGxlLmNvbT4KIAogICAgICAgICBSZXZpZXdlZCBieSBNYWNp ZWoKSW5kZXg6IFdlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckxheWVyLmNwcAo9PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0t LSBXZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJMYXllci5jcHAJKHJldmlzaW9uIDE5NDgxKQorKysg V2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyTGF5ZXIuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0xMjMw LDYgKzEyMzAsMTIgQEAgUmVuZGVyTGF5ZXI6OnVwZGF0ZVNjcm9sbEluZm9BZnRlckxheW91dAog ICAgICAgICAgICAgfQogICAgICAgICB9CiAgICAgfQorICAgIAorICAgIC8vIElmIG92ZXJmbG93 OnNjcm9sbCBpcyB0dXJuZWQgaW50byBvdmVyZmxvdzphdXRvIGEgYmFyIG1pZ2h0IHN0aWxsIGJl IGRpc2FibGVkIChCdWcgMTE5ODUpLgorICAgIGlmIChtX2hCYXIgJiYgbV9vYmplY3QtPmhhc0F1 dG9Ib3Jpem9udGFsU2Nyb2xsYmFyKCkpCisgICAgICAgIG1faEJhci0+c2V0RW5hYmxlZCh0cnVl KTsKKyAgICBpZiAobV92QmFyICYmIG1fb2JqZWN0LT5oYXNBdXRvVmVydGljYWxTY3JvbGxiYXIo KSkKKyAgICAgICAgbV92QmFyLT5zZXRFbmFibGVkKHRydWUpOwogCiAgICAgLy8gU2V0IHVwIHRo ZSByYW5nZSAoYW5kIHBhZ2Ugc3RlcC9saW5lIHN0ZXApLgogICAgIGlmIChtX2hCYXIpIHsK 13112 2007-02-10 17:14:47 -0800 2007-02-10 17:14:47 -0800 test case 11985.html text/html 400 koivisto PGh0bWw+Cjxib2R5PgpTY3JvbGxiYXJzIHNob3VsZCB3b3JrCjxkaXYgaWQ9eCBzdHlsZT0ncG9z aXRpb246cmVsYXRpdmU7IHdpZHRoOjEwMHB4OyBoZWlnaHQ6MTAwcHg7IG92ZXJmbG93OiBzY3Jv bGwnPgo8L2Rpdj4KPHNjcmlwdD4KdmFyIGUgPSBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgneCcp OwovLyBmb3JjZSByZWxheW91dCB0byBjcmVhdGUgc2Nyb2xsYmFycwplLm9mZnNldFRvcDsKLy8g c3dpdGNoIHRvIGF1dG8KZS5zdHlsZS5vdmVyZmxvdyA9ICdhdXRvJzsKLy8gYWRkIHNvbWUgY29u dGVudAplLmlubmVySFRNTCA9ICdhYWFhYWFhYWFhYWFhYWFhYWFhYWFhYWFhYWFhYWFhYWFhPGJy PmE8YnI+YTxicj5hPGJyPmE8YnI+YTxicj5hPGJyPmE8YnI+YTxicj5hPGJyPic7Cjwvc2NyaXB0 Pg==