277599 - embedprompter should wrap text in the check box

Status: RESOLVED FIXED

(Core Graveyard :: Embedding: GTK Widget, defect)

Description

[Christian Persch (GNOME) (away; not receiving bug mail)]

EmbedPrompter doesn't wrap the text in the check box label. That leads to
abominations such as this one:
http://www.cs.vu.nl/~reinout/Veiligheidswaarschuwing.png

Comment 1

[Christian Persch (GNOME) (away; not receiving bug mail)]

Attachment: enable line wrapping in the label

Comment 2

[Christian Persch (GNOME) (away; not receiving bug mail)]

Attachment: screenshot of a wrapped checkbox label with this patch

Comment 3

[Marco Pesenti Gritti]

Comment on attachment 170703 [details] [diff] [review]
enable line wrapping in the label

The visual result is not yet optimal (probably check boxes labels should not be
that long in first place) but it's way better and it make sense to wrap the
checkbox text since we are wrapping the message text.

Comment 4

[Christian :Biesinger (don't email me, ping me on IRC)]

I checked this patch in:
Checking in embedding/browser/gtk/src/EmbedPrompter.cpp;
/cvsroot/mozilla/embedding/browser/gtk/src/EmbedPrompter.cpp,v  <-- 
EmbedPrompter.cpp
new revision: 1.15; previous revision: 1.14
done