Menu

#47 contextmenu display in netscape

0.7.x
closed
None
5
2003-11-29
2003-11-25
No

Hi,
i have noted that the contextmenu was not correctly
displayed in netscape : some texts in the menu were
outside the DIV of the menu.

To correct this, REPLACE line 150 and line 280
(menuobj.innerHTML=which;) by this code :

menuobj.innerHTML=which;
var i;
var tabA = menuobj.getElementsByTagName('A');
var W=0;
for (i=0; i<tabA.length ;i++) {
if (tabA[i].offsetWidth > W) W = tabA
[i].offsetWidth;
}
menuobj.style.width=W;

I joined the screenshots for more understanding.

Matthieu

Discussion

  • Matthieu PAINEAU

    Contextmenu bugfix explanation.

     
  • Matthieu PAINEAU

    Logged In: YES
    user_id=652693

    Sorry, there are mistakes in my last post :
    the filename is tree.js, and it's not line 150 & 280, but it's line
    153 and line 216 (in v1.18 of the file).

     
  • Thomas Bley

    Thomas Bley - 2003-11-29
    • status: open --> closed
     
  • Thomas Bley

    Thomas Bley - 2003-11-29

    Logged In: YES
    user_id=560036

    Your patch has been accepted and added to CVS.

    Check out CVS or fetch a CVS snapshot to see the new code.

    --
    more.groupware developer team

     

Log in to post a comment.