#31879 [Opn->WFx]: [PATCH] function list suggestion does not work

ID: 31879 Updated by: bjori@php.net Reported By: eddie at omegaware dot com -Status: Open +Status: Wont fix Bug Type: Website problem Operating System: All PHP Version: Irrelevant New Comment: This is extremely outdated stuff which isn't really in use anymore. Marking as 'wont fix' Previous Comments: ------------------------------------------------------------------------ [2006-09-28 13:38:58] bjori@php.net Could you have a look Gabor? ------------------------------------------------------------------------ [2005-07-08 03:45:17] eddie at omegaware dot com Here is a weblink to the patch http://www.outoforder.cc/downloads/patches/quickref.diff As one of the previous comments stated, the current function suggestion code doesn't work correctly in IE. This patch provides that fix. This has been tested on IE 5.5, IE 6, Opera, Firefox, Mozilla 1.6, Konqueror, and Opera. ------------------------------------------------------------------------ [2005-07-08 03:36:11] eddie at omegaware dot com Index: originalafter.js =================================================================== RCS file: /repository/phpdoc/scripts/quickref/originalafter.js,v retrieving revision 1.16 diff -u -3 -p -r1.16 originalafter.js --- originalafter.js 11 Feb 2005 12:47:12 -0000 1.16 +++ originalafter.js 8 Jul 2005 01:35:16 -0000 @@ -20,14 +20,21 @@ fh_EDropDownChange(); // Layer setup ---------------------------------------------------------------- - +var funchelper = _d.createElement('div'); +funchelper.style.backgroundColor='white'; +funchelper.style.border='1px solid black'; +funchelper.style.top='90px'; if (_d.all && (isnotopera=(navigator.userAgent.toLowerCase().indexOf("opera")== -1))) { - width="width:165px"; + funchelper.style.width='165px'; } else { isnotopera=true; - width="min-width:155px"; + funchelper.style.width='155px'; } -var funchelper = _d.createElement('div'); -funchelper.setAttribute('style', 'background-color: white; border: 1px solid bl ack; top: 90px;'+width+'; padding: 4px; font-size: 9px; display:none; position:a bsolute;'); +funchelper.style.padding='4px'; +funchelper.style.fontSize='9px'; +funchelper.style.display='none'; +funchelper.style.position='absolute'; + var elems = _d.getElementsByTagName("*"); for (var i = 0; i < elems.length; i++) { if (elems[i].tagName.toLowerCase() == 'body') { ------------------------------------------------------------------------ [2005-07-07 13:03:34] goba@php.net Hi Eddie! Unless you provide us with a diff, it is not likely that we are going to incorporate your changes. Note that the current function suggestion works perfectly in quite a few browsers (I have just tested in Firefox). ------------------------------------------------------------------------ [2005-07-07 00:45:23] eddie at omegaware dot com What is the status on this? Has anyone had a change to incorporate my changes into CVS so that this useful feature can be added and enabled on php.net? ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/31879 -- Edit this bug report at http://bugs.php.net/?id=31879&edit=1