4888 2005-09-08 11:38:05 -0700 Missing support for many window.frameElement properties (Frame size calculation at SoftwarePlanner broken) 2007-12-29 11:09:11 -0800 1 1 1 Unclassified WebKit Frames 420+ Mac OS X 10.4 RESOLVED INVALID http://www.pragmaticsw.com/SoftwarePlanner.asp HasReduction P2 Normal --- 6402 1 ap mjs ian oldest_to_newest 19100 0 ap 2005-09-08 11:38:05 -0700 A project management suite called Software Planner is not correctly rendered by current versions of Safari. It used to work correctly with WebKit up to 312, but doesn't work (in different ways) with 412.2 or ToT. Steps to reproduce: 1. Go to <http://www.pragmaticsw.com/SoftwarePlanner.asp> 2. Click on "Try Sample Database" 3. Click "Logon" Results: - (stock 1.3 and earlier): works correctly - (stock 2.0, 2.0.1 or 1.3.1): the contents briefly renders correctly, then the left frame expands, completely covering the right one - (ToT): the contents briefly renders correctly, then the left frame expands, taking half of the page. Firefox works with this site OK. 19242 1 3831 pg 2005-09-09 14:52:12 -0700 Created attachment 3831 Test case Wait 2 seconds to see the right pane disappear. Software Planner miscomputes frameset.cols, replacing a correct '230, *' with 'NaN, *'. Firefox ignores it, so should Safari. 19255 2 ap 2005-09-10 00:58:27 -0700 (In reply to comment #1) Thank you! There's still a slight weirdness - Safari 1.2.4 fails on this test case, but not on the actual SoftwarePlanner page. Could there be a second problem, which makes frameset.cols get miscomputed on newer versions of WebKit? I'll leave the NeedsReduction keyword there for now... 19270 3 pg 2005-09-10 01:34:01 -0700 I don't have Safari 1.2.4 so I can't test. But you can :) Save the page with firefox, then open psDALeft.htm and comment out line 486 window.frameElement.parentElement.cols = (window.frameElement.width + 10) + ', *'; This removes the bug. Both Safari & Firefox show window.frameElement.width as undefined. Maybe Safari 1.2.4 has a correct value ? 19275 4 ap 2005-09-10 04:43:42 -0700 (In reply to comment #3) Safari 1.2.4 simply did't get to execute this code at all (I have replaced it with a plain alert("test"), and got nothing). I don't understand why, but it seems totally unrelated. Another mistery: on ToT, the test case works OK (I have checked, window.frameElement.width is still undefined) - but the actual page resizes the frame to 50%... 19278 5 pg 2005-09-10 07:38:38 -0700 > Safari 1.2.4 simply did't get to execute this code at all Maybe some Javascript fails before it. Does the JS console show anything ? > Another mistery: on ToT, the test case works OK (I have checked, window.frameElement.width is still > undefined) - but the actual page resizes the frame to 50%... I just tried the test case in ToT and at first glance it does work. However, the JS in the test case is NOT executed. Add an alert() and nothing pops up ! So, copy javascript:alert(document.getElementsByTagName('FRAMESET')[0].cols = 'NaN, *') in the address bar and launch it to see the frames resized like in the actual page. 19288 6 3845 ap 2005-09-10 11:23:03 -0700 Created attachment 3845 Incomplete Test 19289 7 ap 2005-09-10 11:23:47 -0700 (In reply to comment #5) I see... To work in ToT, the test needs to be modified a bit. However, it looks like Firefox also doesn't ignore NaN! After 2 seconds, the right frame expands to 100%. There must be something else in the actual page that makes it work in Firefox... 19451 8 ap 2005-09-12 02:48:54 -0700 There are probably at least two issues here: window.frameElement.width is not undefined in WinIE - it's 230, so the +10 part does work as intended. 20579 9 4032 ap 2005-09-24 12:51:53 -0700 Created attachment 4032 frameElement tests 20581 10 ap 2005-09-24 12:57:26 -0700 Ok, so Firefox renders the page fine because it doesn't handle window.frameElement.parentElement.cols at all: try { window.frameElement.parentElement.cols; } catch (e) { alert('Exception: ' + e.description); } prints 'Exception: undefined'. This leaves us with a single WebKit issue - lack of support for window.frameElement.width (and some other frame properties supported by WinIE, see the attached test case for details). 65612 11 ap 2007-12-29 11:09:11 -0800 frameElement.width was implemented in bug 6402, so the actual problem is fixed. It doesn't look like keeping this bug is doing us any good, closing. 3831 2005-09-09 14:52:12 -0700 2005-09-10 11:23:03 -0700 Test case frame.html text/html 261 pg PGh0bWw+Cgk8ZnJhbWVzZXQgY29scz0iMjMwLCoiIHN0eWxlPSdib3JkZXI6IHNvbGlkIDFweCBs aW1lOyc+CgkJPGZyYW1lIHNyYz0iYWJvdXQ6IGJsYW5rIj4KCQk8ZnJhbWUgc3JjPSJhYm91dDog YmxhbmsiPgoJPC9mcmFtZXNldD4KCgk8c2NyaXB0PgoJCXNldFRpbWVvdXQoZnVuY3Rpb24gKCkg eyBkb2N1bWVudC5nZXRFbGVtZW50c0J5VGFnTmFtZSgnRlJBTUVTRVQnKVswXS5jb2xzID0gJ05h TiwgKicgfSwgMjAwMCkKCTwvc2NyaXB0Pgo8L2h0bWw+ 3845 2005-09-10 11:23:03 -0700 2005-09-24 12:51:53 -0700 Incomplete Test SP-test.html text/html 258 ap PGh0bWw+Cgk8c2NyaXB0PgoJCXNldFRpbWVvdXQoZnVuY3Rpb24gKCkgeyBkb2N1bWVudC5nZXRF bGVtZW50c0J5VGFnTmFtZSgnRlJBTUVTRVQnKVswXS5jb2xzID0gJ05hTiwgKicgfSwgMjAwMCkK CTwvc2NyaXB0PgoJPGZyYW1lc2V0IGNvbHM9IjIzMCwqIiBzdHlsZT0nYm9yZGVyOiBzb2xpZCAx cHggbGltZTsnPgoJCTxmcmFtZSBzcmM9ImFib3V0OmJsYW5rIj4KCQk8ZnJhbWUgc3JjPSJhYm91 dDpibGFuayI+Cgk8L2ZyYW1lc2V0Pgo8L2h0bWw+ 4032 2005-09-24 12:51:53 -0700 2005-09-24 12:51:53 -0700 frameElement tests frameElement.zip application/octet-stream 2387 ap UEsDBAoAAAAAALexODMAAAAAAAAAAAAAAAANABAAZnJhbWVFbGVtZW50L1VYDAB/qzVDWZc1Q/UB 9QFQSwMEFAAIAAgAUb44MwAAAAAAAAAAAAAAABYAEABmcmFtZUVsZW1lbnQvbGVmdC5odG1sVVgM ABGuNUMKrjVD9QH1AZWVQU/CMBSAz+xX9DaNyXrHSSLBhANe1MRzaR+02rVL+3Cg8b/bDZDF2FFO 7d779r1le+srJTAxyUpUqGGygBWWdL/PSnrILa3YhcVzp2qcZOh25CsbfTBHPDpyR/J7jhumiQO/ 0ejHpFy6SX6bjdr0Tcg3ygjbFCvHKnjQUIHBu5zckH/iIZqX06dztxfcGgzrzPLNkO4vd6H+tUud k++pVHV3MbVOgIuKe0yqVluznoHnUecRSBVWzK2VmYNay/jr7UOXiV+VQHnG2zGpWhP2UV+bTBbZ J/Dqc0B2AFKF4cexWiuzjhp/iWSli3/okOtrMkrJMwCRiPWY0soLU1QqFPR21bZwRRvr3r20NWUb lNZRIbHS1MEKHBgO1C7fgKOnXZWC+Tobfjg53DPyom5pBvukuaRDODNz9gHzl8dF/Jc+MalaoTxb ahBR5xFIFSo/O6c8IenSx3A4q4Uy8cbuMalaF0bE7hkZxq0npCfNRuJwJBeNUwhXoch1CH8TzpBL cgXb63bK/KHyhy2HGpU1Y9LWg20hYD+YQiwIvsPYOg6qknaT6wdQSwcIcR6zl6gBAADjBgAAUEsD BAoAAAAAAFW+ODMAAAAAAAAAAAAAAAAJABAAX19NQUNPU1gvVVgMABGuNUMRrjVD9QH1AVBLAwQK AAAAAABVvjgzAAAAAAAAAAAAAAAAFgAQAF9fTUFDT1NYL2ZyYW1lRWxlbWVudC9VWAwAEa41QxGu NUP1AfUBUEsDBBQACAAIAFG+ODMAAAAAAAAAAAAAAAAhABAAX19NQUNPU1gvZnJhbWVFbGVtZW50 Ly5fbGVmdC5odG1sVVgMABGuNUMKrjVD9QH1AWNgFWNnYGLABCAxTiA2AmIFKD8IJBHiGhGiGJSc gUUPGAAAUEsHCEbZ8A0jAAAAUgAAAFBLAwQUAAgACAAwvjgzAAAAAAAAAAAAAAAAFgAQAGZyYW1l RWxlbWVudC9tYWluLmh0bWxVWAwAEa41Q8ytNUP1AfUBTY5BDsIgEEX3nGLCpokx0i7cKLDzIFim hWQoBiZRb2+lGl395L+Zn6cDJ7JCB3R+DY5MaKfiEl4IEy6s1dYJAbr1FRmuuXgsRg4SxkzVyGPf 73cSKj8JTbfhE9RM0cNwe5w7K+DzD8mVOS736Dm0hVpGIwknPrxd5N9lIyXO4Ye0+kqstqrJvwBQ SwcIc8v57IkAAADDAAAAUEsDBBQACAAIADC+ODMAAAAAAAAAAAAAAAAhABAAX19NQUNPU1gvZnJh bWVFbGVtZW50Ly5fbWFpbi5odG1sVVgMABGuNUPMrTVD9QH1AWNgFWNnYGLABCAxTiA2AmIFKD8I JBHiGhGiGJScgUUPGAAAUEsHCEbZ8A0jAAAAUgAAAFBLAwQUAAgACAAIvjgzAAAAAAAAAAAAAAAA FwAQAGZyYW1lRWxlbWVudC9yaWdodC5odG1sVVgMABGuNUN/rTVD9QH1AY3SwWrDMAwA0Hu+ol+Q tofRMeIcRgsZtJdtkMPYwbGVRMORwFYWuq9flpxFdrKNn2zLUtHLEMqsaNjfy6xGernsIqQxSHra FU0sswnJ85S30Q5wCTAAifng5gucfGogd0wyj2d24xIwkocWCfxWRL1sbV+wLJ45eohGRYGpO0Ny uhhs7JAqwK6XLVWjl94cVUXzXD+D+BUS/oBpbUigsuQih4DUGTsK6yw6E6CV/K98qurXvE7HR5VM S1IPh4NeGEuV/Ybq/XbdeLvHZJsAfoNhOv8X3uY+xOvcN0biqMMI1t/fxAqYwNbPv7fQYr92dbFf m/wXUEsHCDRd8wT1AAAA7AIAAFBLAwQUAAgACAAIvjgzAAAAAAAAAAAAAAAAIgAQAF9fTUFDT1NY L2ZyYW1lRWxlbWVudC8uX3JpZ2h0Lmh0bWxVWAwAEa41Q3+tNUP1AfUBY2AVY2dgYsAEIDFOIDYC YgUoPwgkEeIaEaIYlJyBRQ8YAABQSwcIRtnwDSMAAABSAAAAUEsBAhUDCgAAAAAAt7E4MwAAAAAA AAAAAAAAAA0ADAAAAAAAAAAAQO1BAAAAAGZyYW1lRWxlbWVudC9VWAgAf6s1Q1mXNUNQSwECFQMU AAgACABRvjgzcR6zl6gBAADjBgAAFgAMAAAAAAAAAABApIE7AAAAZnJhbWVFbGVtZW50L2xlZnQu aHRtbFVYCAARrjVDCq41Q1BLAQIVAwoAAAAAAFW+ODMAAAAAAAAAAAAAAAAJAAwAAAAAAAAAAED9 QTcCAABfX01BQ09TWC9VWAgAEa41QxGuNUNQSwECFQMKAAAAAABVvjgzAAAAAAAAAAAAAAAAFgAM AAAAAAAAAABA/UFuAgAAX19NQUNPU1gvZnJhbWVFbGVtZW50L1VYCAARrjVDEa41Q1BLAQIVAxQA CAAIAFG+ODNG2fANIwAAAFIAAAAhAAwAAAAAAAAAAECkgbICAABfX01BQ09TWC9mcmFtZUVsZW1l bnQvLl9sZWZ0Lmh0bWxVWAgAEa41QwquNUNQSwECFQMUAAgACAAwvjgzc8v57IkAAADDAAAAFgAM AAAAAAAAAABApIE0AwAAZnJhbWVFbGVtZW50L21haW4uaHRtbFVYCAARrjVDzK01Q1BLAQIVAxQA CAAIADC+ODNG2fANIwAAAFIAAAAhAAwAAAAAAAAAAECkgREEAABfX01BQ09TWC9mcmFtZUVsZW1l bnQvLl9tYWluLmh0bWxVWAgAEa41Q8ytNUNQSwECFQMUAAgACAAIvjgzNF3zBPUAAADsAgAAFwAM AAAAAAAAAABApIGTBAAAZnJhbWVFbGVtZW50L3JpZ2h0Lmh0bWxVWAgAEa41Q3+tNUNQSwECFQMU AAgACAAIvjgzRtnwDSMAAABSAAAAIgAMAAAAAAAAAABApIHdBQAAX19NQUNPU1gvZnJhbWVFbGVt ZW50Ly5fcmlnaHQuaHRtbFVYCAARrjVDf601Q1BLBQYAAAAACQAJAN0CAABgBgAAAAA=