Yes, it would be better to just use something else for a separator besides
<hr> if that fixes the IE7 bug.
I'll try your solution.



On Fri, May 8, 2009 at 1:21 PM, AndrĂ© Bargull <[email protected]> wrote:

> 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
>>
>>


-- 
Henry Minsky
Software Architect
[email protected]

Reply via email to