110303 - no "ok" in insert link dialog

Status: VERIFIED FIXED

(SeaMonkey :: Composer, defect)

Description

[Judson Valeski]

using today's trunk build there is no "ok" button when you open the "insert
link" dialog (toolbar or menu item or ctrl-l) (note, my dialog is set to
advanced mode by default).

steps to repro:
1. create a new document in composer
2. add some text.
3. select some text.
4. select "link" from the toolbar

results: the dialog comes up, but, there is no 'ok' , or cancel button at the
bottom.

expected results: the dialog should have those buttons.

after clicking the advanced edit button, the ok and cancel buttons show up.

Comment 1

[Judson Valeski]

I meant the "fewer properties" button, not the advanced edit button; sorry.

Comment 2

[Charles Manske]

Click on the "More / Fewer" button. Do the Ok, Cancel buttons appear?

Comment 3

[Judson Valeski]

yes

Comment 4

[Charles Manske]

This is a XUL layout problem. Seems I see this kind of problem more often since 
conversion to "<dialog" XUL tag.

Comment 5

[sujay]

heads up to Tucson.

Comment 6

[Charles Manske]

I see this when I start the dialog in the "more" mode, with larger size, but 
only the first time the dialog is used. I also see it if the default state is
"less" (smaller), and I click on the "More" button the first time.

Comment 7

[Charles Manske]

I've done everthing I can think of, and these seem to be an intrinisic sizing
problem. Please investigate!

Comment 8

[sujay]

*** Bug 111169 has been marked as a duplicate of this bug. ***

Comment 9

[Charles Manske]

I think the problem is that the "rows" attribute is not used to correctly predict
the height the tree elements (lists) will occupy.
Adding style to set the min-height seems to fix this.
Taking the bug back.

Comment 10

[Charles Manske]

Attachment: Add min-height to be sure height of lists are calculated correctly

Comment 11

[kinmoz]

Comment on attachment 58730 [details] [diff] [review]
Add min-height to be sure height of lists are calculated correctly

sr=[e-mail]

Comment 12

[Syd Logan]

r=syd

Comment 13

[Charles Manske]

checked in

Comment 14

[michael wendell]

Verified on build 200112603.