From 02c618aacd7f1b4683b73e93fa063d4db9e02fcd Mon Sep 17 00:00:00 2001 From: "mkanat%bugzilla.org" <> Date: Sat, 9 Feb 2008 05:18:59 +0000 Subject: Bug 388723: Make the browser wrap comments on display in addition to the normal wrapping, to support CJK languages that have no spaces. Patch By Max Kanat-Alexander r=himorin, r=glob --- skins/standard/IE-fixes.css | 5 +++++ skins/standard/global.css | 21 +++++++++++++++++++-- template/en/default/attachment/midair.html.tmpl | 4 +++- template/en/default/bug/comments.html.tmpl | 3 ++- template/en/default/bug/process/midair.html.tmpl | 4 +++- template/en/default/pages/linked.html.tmpl | 4 ++-- 6 files changed, 34 insertions(+), 7 deletions(-) diff --git a/skins/standard/IE-fixes.css b/skins/standard/IE-fixes.css index b0f34141d..bfd525ba2 100755 --- a/skins/standard/IE-fixes.css +++ b/skins/standard/IE-fixes.css @@ -13,6 +13,11 @@ * Contributor(s): Marc Schumann */ +.bz_comment_text, .uneditable_textarea { + white-space: pre; + word-wrap: break-word; +} + #footer #useful-links li { padding-bottom: 0.8ex; } diff --git a/skins/standard/global.css b/skins/standard/global.css index edf542510..423e398b4 100644 --- a/skins/standard/global.css +++ b/skins/standard/global.css @@ -243,6 +243,22 @@ div#docslinks { margin-bottom: 2em; } +/* The rules for these classes make international text wrap correctly, + even for languages like Japanese that have no spaces. */ +.bz_comment_text, .uneditable_textarea { + font-family: monospace; + /* Note that these must all be on separate lines or they stop + working in Konqueror. */ + white-space: pre-wrap; /* CSS 3 & 2.1 */ + white-space: -moz-pre-wrap; /* Gecko */ + white-space: -pre-wrap; /* Opera 4-6 */ + white-space: -o-pre-wrap; /* Opera 7 */ +} + +.bz_comment_text { + width: 50em; +} + .bz_first_comment { } @@ -257,6 +273,8 @@ div#docslinks { span.quote { color: #65379c; + /* Make quoted text not wrap. */ + white-space: pre; } table#flags th, @@ -318,8 +336,7 @@ dl dl > dt { /* For bug fields */ .uneditable_textarea { - white-space: pre; - font-family: monospace; + width: 30em; } div.user_match { diff --git a/template/en/default/attachment/midair.html.tmpl b/template/en/default/attachment/midair.html.tmpl index 8cde9f2f5..f0883b55b 100644 --- a/template/en/default/attachment/midair.html.tmpl +++ b/template/en/default/attachment/midair.html.tmpl @@ -50,7 +50,9 @@ [% IF cgi.param("comment") %]

Your comment was:
-

[% cgi.param("comment") FILTER wrap_comment FILTER html %]
+
+    [% cgi.param("comment") FILTER wrap_comment FILTER html %]
+  

[% END %] diff --git a/template/en/default/bug/comments.html.tmpl b/template/en/default/bug/comments.html.tmpl index 1fc5d46d3..da3fe3a9d 100644 --- a/template/en/default/bug/comments.html.tmpl +++ b/template/en/default/bug/comments.html.tmpl @@ -216,7 +216,8 @@ [% ELSE %] [% wrapped_comment = comment.body FILTER wrap_comment %] [% END %] - +
   [%- wrapped_comment FILTER quoteUrls(bug.bug_id) -%]
 
diff --git a/template/en/default/bug/process/midair.html.tmpl b/template/en/default/bug/process/midair.html.tmpl index 31dbc7b3c..d7e980e8c 100644 --- a/template/en/default/bug/process/midair.html.tmpl +++ b/template/en/default/bug/process/midair.html.tmpl @@ -66,7 +66,9 @@ [% IF cgi.param("comment") %]

Your comment was:
-

[% cgi.param("comment") FILTER wrap_comment FILTER html %]
+
+    [% cgi.param("comment") FILTER wrap_comment FILTER html %]
+  

[% END %] diff --git a/template/en/default/pages/linked.html.tmpl b/template/en/default/pages/linked.html.tmpl index a5e40d122..52b1735f6 100644 --- a/template/en/default/pages/linked.html.tmpl +++ b/template/en/default/pages/linked.html.tmpl @@ -30,7 +30,7 @@

-

+
 [%- cgi.param("text") FILTER wrap_comment FILTER quoteUrls FILTER html -%]
 

@@ -45,7 +45,7 @@

-

+
 [%- cgi.param("text") FILTER wrap_comment FILTER quoteUrls -%]
 

-- cgit v1.2.3-24-g4f1b