• Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • core
    • None
    • Platform: All, OS: All

      Subversion commit messages do not wrap if they do not contain line breaks.
      When rendered as HTML, all of the spaces are replaced with  , which
      prevents the wrapping. I don't think this is particularly useful behaviour
      since it means lots of horizontal scrolling when the message is long.

          [JENKINS-1031] Log messages on changes page do not wrap.

          dwdyer added a comment -

          Fixed in version 1.36 of Util.java. Modified the escape(String) method so that
          it doesn't replace spaces with HTML non-breaking space entities. Should be in
          Hudson 1.159.

          dwdyer added a comment - Fixed in version 1.36 of Util.java. Modified the escape(String) method so that it doesn't replace spaces with HTML non-breaking space entities. Should be in Hudson 1.159.

          I agree that we should render Subversion commit messages to allow wrapping, but
          I have a problem with this change.

          I think Util.espcae should remain faithful conversion from ascii text to HTML.
          Not converting whitespace to   means consective whitespaces will be
          combined into one.

          I suggest we handle this differently. We should convert like:

          'abc def' -> 'abc def'
          'abc def' -> 'abc  def'
          'abc def' -> 'abc   def'
          'abc def' -> 'abc    def'
          'abc def' -> 'abc     def'

          In this way we can allow line wraps but without incorrectly trimming whitespaces.

          Kohsuke Kawaguchi added a comment - I agree that we should render Subversion commit messages to allow wrapping, but I have a problem with this change. I think Util.espcae should remain faithful conversion from ascii text to HTML. Not converting whitespace to   means consective whitespaces will be combined into one. I suggest we handle this differently. We should convert like: 'abc def' -> 'abc def' 'abc def' -> 'abc  def' 'abc def' -> 'abc   def' 'abc def' -> 'abc    def' 'abc def' -> 'abc     def' In this way we can allow line wraps but without incorrectly trimming whitespaces.

          dwdyer added a comment -

          OK. I'll take a look at doing it the way that you suggest when I get some time
          later today.

          dwdyer added a comment - OK. I'll take a look at doing it the way that you suggest when I get some time later today.

          dwdyer added a comment -

          I've just checked in a change (version 1.37 of Util.java) that implements the
          space-escaping as suggested. Tested locally and it seems fine.

          dwdyer added a comment - I've just checked in a change (version 1.37 of Util.java) that implements the space-escaping as suggested. Tested locally and it seems fine.

            dwdyer dwdyer
            dwdyer dwdyer
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Created:
              Updated:
              Resolved: