The current change breaks the css-layout, doesn't it? Because the css-selectors won't work anymore ("#lzcontextmenu a" resp. "#lzcontextmenu a:hover", see "LzSprite.js").

Another solution would be to use styled div-tags, like:
<div class="separator"></div>

With the proper layout:
// Firefox
#lzcontextmenu div.separator {
    border-top: 1px solid #4b4b4b;
    border-left: 1px solid #4b4b4b;
    border-right: 1px solid #d3d3d3;
    border-bottom: 1px solid #d3d3d3;
    margin: 6px 0px;
}

// Safari
#lzcontextmenu div.separator {
    border-top: 1px solid #9a9a9a;
    border-left: 1px solid #9a9a9a;
    border-right: 1px solid #eeeeee;
    border-bottom: 1px solid #eeeeee;
    margin: 6px 0px;
}

// Opera
#lzcontextmenu div.separator {
    border-top: 1px solid #4c4c4c;
    border-left: 1px solid #4c4c4c;
    border-right: 1px solid #cccccc;
    border-bottom: 1px solid #cccccc;
    margin: 10px 0px;
}

// Internet Explorer
#lzcontextmenu div.separator {
    border-top: 1px solid #808080;
    border-left: none;
    border-right: none;
    border-bottom: 1px solid #d4d0c8;
    margin: 7px 0px;
}


On 5/7/2009 6:53 PM, Henry Minsky wrote:
Change 20090507-hqm-x by [email protected] on 2009-05-07 12:51:20 EDT
    in /Users/hqm/openlaszlo/trunk-diamond
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: fix for  DHTML IE: Context menus too wide

New Features:

Bugs Fixed: LPP-8137

Technical Reviewer: max
QA Reviewer: andre
Doc Reviewer: (pending)

Documentation:

Release Notes:

Details:

+ enclose DHTML context menu within a <table>, to prevent the <hr> from expanding to
the width of the canvas in IE7.


Tests:

tested in FF/OSX, and IE7/WINXP  DHTML
test/contextmenu/ie7.lzx

Files:
A      test/contextmenu/ie7.lzx
M      WEB-INF/lps/lfc/kernel/dhtml/LzContextMenuKernel.lzs


Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20090507-hqm-x.tar

Reply via email to