Bug 245282 : Flags should use nonbreaking hyphens or nowrap or somehow prevent hyphen...

Patch by LpSolit@gmail.com    r=wurblzap    a=myk

diff --git a/Bugzilla/Template.pm b/Bugzilla/Template.pm
index cddd33ba47ffd6d5dac71dbc6cf76ef3ded9d150..cb034478d448b1e1c1aa7208a16eea56b072f3cc 100644 (file)
--- a/Bugzilla/Template.pm
+++ b/Bugzilla/Template.pm
@@ -249,6 +249,14 @@ sub create {
                 return $var;
             },
 
+            # Prevents line break on hyphens and whitespaces.
+            no_break => sub {
+                my ($var) = @_;
+                $var =~ s/ /\ /g;
+                $var =~ s/-/\‑/g;
+                return $var;
+            },
+
             xml => \&Bugzilla::Util::xml_quote ,
 
             # This filter escapes characters in a variable or value string for

diff --git a/checksetup.pl b/checksetup.pl
index 9615beb73091c1482fbc917b16ebd3ffa19a8fd5..a01d0962a0ca86191b902a243c56795ce49341b8 100755 (executable)
--- a/checksetup.pl
+++ b/checksetup.pl
@@ -1249,6 +1249,7 @@ END
                 obsolete => sub { return $_; },
                 js => sub { return $_; },
                 html_linebreak => sub { return $_; },
+                no_break => sub { return $_; },
                 url_quote => sub { return $_; },
                 xml => sub { return $_; },
                 quoteUrls => sub { return $_; },

diff --git a/t/004template.t b/t/004template.t
index 86b505c285ea245c59a977d818ecfeee379d3553..f35b5342568203fbd947440ccdf2797621ab313c 100644 (file)
--- a/t/004template.t
+++ b/t/004template.t
@@ -93,6 +93,7 @@ foreach my $include_path (@include_paths) {
         FILTERS =>
         {
             html_linebreak => sub { return $_; },
+            no_break => sub { return $_; } ,
             js        => sub { return $_ } ,
             inactive => [ sub { return sub { return $_; } }, 1] ,
             closed => [ sub { return sub { return $_; } }, 1] ,

diff --git a/template/en/default/admin/flag-type/list.html.tmpl b/template/en/default/admin/flag-type/list.html.tmpl
index 44cbf36c29ab5bccba75e1fc4d5cc35fb89d051c..9638df8914909ef7bd42c98dc271db41e0384b88 100644 (file)
--- a/template/en/default/admin/flag-type/list.html.tmpl
+++ b/template/en/default/admin/flag-type/list.html.tmpl
@@ -93,7 +93,7 @@
     [% FOREACH type = types %]
 
       <tr class="[% IF type.is_active %]active[% ELSE %]inactive[% END %]">
-        <td><a href="editflagtypes.cgi?action=edit&amp;id=[% type.id %]">[% type.name FILTER html %]</a></td>
+        <td><a href="editflagtypes.cgi?action=edit&amp;id=[% type.id %]">[% type.name FILTER html FILTER no_break %]</a></td>
         <td>[% type.description FILTER html %]</td>
         <td>
           <a href="editflagtypes.cgi?action=copy&amp;id=[% type.id %]">Copy</a>

diff --git a/template/en/default/attachment/list.html.tmpl b/template/en/default/attachment/list.html.tmpl
index d5741d6a2cd2a5be4c110999fa946b7b1f14080e..41115ab9cc5b228ff2f430cb64efb0439a77e855 100644 (file)
--- a/template/en/default/attachment/list.html.tmpl
+++ b/template/en/default/attachment/list.html.tmpl
@@ -59,7 +59,7 @@
                 [% IF flag.setter %]
                   [% flag.setter.nick FILTER html %]:
                 [% END %]
-                [%+ flag.type.name FILTER html %][% flag.status %]
+                [%+ flag.type.name FILTER html FILTER no_break %][% flag.status %]
                 [%+ IF flag.status == "?" && flag.requestee %]
                   ([% flag.requestee.nick FILTER html %])
                 [% END %]<br>

diff --git a/template/en/default/flag/list.html.tmpl b/template/en/default/flag/list.html.tmpl
index 08f8c5547ffd1199989acdc7ebb80fc8cae763fb..f660b15cada774b5d54ca80b8d6a8cb869c8d73b 100644 (file)
--- a/template/en/default/flag/list.html.tmpl
+++ b/template/en/default/flag/list.html.tmpl
@@ -98,7 +98,7 @@
           [% flag.setter.nick FILTER html %]:
         </td>
         <td>
-          [% type.name FILTER html %]
+          [% type.name FILTER html FILTER no_break %]
         </td>
         <td>
           <select id="flag-[% flag.id %]" name="flag-[% flag.id %]" 
@@ -137,7 +137,7 @@
     [% IF (!type.flags || type.flags.size == 0) && type.is_active %]
       <tr>
         <td>&nbsp;</td>
-        <td>[% type.name FILTER html %]</td>
+        <td>[% type.name FILTER html FILTER no_break %]</td>
         <td>
           <select id="flag_type-[% type.id %]" name="flag_type-[% type.id %]" 
                   onchange="toggleRequesteeField(this);">
@@ -172,7 +172,7 @@
         [% separator_displayed = 1 %]
     [% END %]
     <tr>
-      <td colspan="2">addl. [% type.name FILTER html %]</td>
+      <td colspan="2">addl. [% type.name FILTER html FILTER no_break %]</td>
       <td>
         <select id="flag_type-[% type.id %]" name="flag_type-[% type.id %]" 
                 onchange="toggleRequesteeField(this);">