if you have many anchors in a page, the listbox text overflows the area allocated to it in the link properties dialog.
Thanks for filing this for me! I've been distracted and forgot to do it since we talked.
cc'ing some folks who might know more about what's going on.
*** Bug 112895 has been marked as a duplicate of this bug. ***
*** Bug 103621 has been marked as a duplicate of this bug. ***
note to self - construct a simpler test case xul file.
Note: The XUL for the Link Properties dialog "listbox" is: <tree rows="4" class="list" id="NamedAnchorList" onselect="SelectNamedAnchor()" style="min-height: 4em"> <treecolgroup><treecol flex="1"/></treecolgroup> The <treechildren> ... <treecell> members are added at runtime. The basic issue is that neither the "rows" attribute nor the setting of min-height by the style attribute seem to work. The actual height of the box around the entire tree seems to be about 2.5em (Modern skin), thus the last 2 items in the list draw outside the enclosing box. Note that the Spell Check dialog also has this problem (there, rows="8" and I also set min-height on the tree at 8em), this is a bugscape bug.
improving the summary
this was also reproduced on Mac OS X build 2001121805.80.
Modified summary to add "Spelling" dialog, since that shows the same problem
*** Bug 111312 has been marked as a duplicate of this bug. ***
Setting dependancy. I belive this issue should disappear when listbox is implemented.
Bug 111312 for Mail Client is marked as duplicate of this one. This is just a note not to forget to test Mail also when verify it from Browser.
There are old styles in EditorDialogs.css for tree.list which set borders and dimensions. If you remove them then the dialog displays perfectly.
Thanks, Neil! While we should convert to the new "list" widget when available, we have a fix now for this problem.
Created attachment 63604 [details] [diff] [review] Remove bad CSS rules and extra style attributes in dialogs Note that in the spelling dialog, there's a problem with sizing the dialog if we simply remove the CSS in editorDialog.css -- buttons on bottom are cut off. Changing number of rows to 6 fixes this (that is what we had originally anyway).
Neil: review please?
Created attachment 63605 [details] [diff] [review] Updated patch: Remove style in both lists in EdLinkProps.xul
Comment on attachment 63605 [details] [diff] [review] Updated patch: Remove style in both lists in EdLinkProps.xul looks good to me. Removing inline styles is always a plus! r=andreww
Comment on attachment 63605 [details] [diff] [review] Updated patch: Remove style in both lists in EdLinkProps.xul sr=hewitt
Verified on trunk build 2002013003. If anyone is still seeing this problem, please reopen this bug.