#8427 closed defect (fixed)
[regression] Menu: menu often has border between label and right arrow (safari, chrome)
Reported by: | bill | Owned by: | liucougar |
---|---|---|---|
Priority: | high | Milestone: | 1.3 |
Component: | Dijit | Version: | 1.2.3 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description (last modified by )
On safari, in test_Tree.html, and in the drop down menus from the MenuBar in test_Menu.html, the menu shows a vertical bar between the labels and the arrows.
Looks OK on FF.
This started w/[16168] but it's actually due to a bug in safari. Test case from Cougar:
<table style="border:1px solid #7EABCD;border-collapse:collapse;"> <tbody><tr><td></td><td colspan="2">one</td><td> </td> </tr></tbody></table>
In order to reproduce this bug, it has to meet the following:
- border-collapse is set to collapse
- a td with colspan=2
- there is a td before the colspan=2 td
Attachments (1)
Change History (7)
Changed 13 years ago by
Attachment: | treeMenu.png added |
---|
comment:1 Changed 13 years ago by
Description: | modified (diff) |
---|---|
Summary: | Menu: menu on tree test page has vertical bar (safari) → Menu: menu often has border between label and right arrow (safari) |
comment:2 Changed 13 years ago by
Filed https://bugs.webkit.org/show_bug.cgi?id=23378 against webkit.
comment:3 Changed 13 years ago by
Milestone: | tbd → 1.3 |
---|---|
Owner: | set to liucougar |
Summary: | Menu: menu often has border between label and right arrow (safari) → [regression] Menu: menu often has border between label and right arrow (safari, chrome) |
I don't want to release 1.3 w/this regression (even though it's due to a webkit bug). Can be worked around by:
- don't use colspan=2, just always leave column for accelerators
- don't use colspan=2 for webkit (note that problem is safari and chrome so the correct check is for webkit)
- some clever CSS change to make the problem go away
comment:4 Changed 13 years ago by
Priority: | normal → high |
---|
comment:5 Changed 13 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:6 Changed 13 years ago by
Note: See
TracTickets for help on using
tickets.
screen shot