ICEfaces
  1. ICEfaces
  2. ICE-1351

newline emitted after </input>

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.5.3
    • Fix Version/s: 1.6.1
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      firefox 2.0/winxp, facelets

      Description

      Icefaces renderer appears to append a newline character after </input> element resulting in gaps between commandButtons rendered next to each other. Not entirely sure where this happens but suspecting DOMUtils.printNode method.

      Check out screens (before and after generated source was manually-edited to remove the newline). also attaching sample xhtml file and button image

        Activity

        Hide
        yip.ng added a comment -

        Test webapp.

        Note: before running the webapp, add the relevant jars to the WEB-INF/lib dir, including icefaces.jar, where the code change will be compiled into.

        Show
        yip.ng added a comment - Test webapp. Note: before running the webapp, add the relevant jars to the WEB-INF/lib dir, including icefaces.jar, where the code change will be compiled into.
        Hide
        yip.ng added a comment -

        Fixed by adding code to not emit the newline. Test webapp attached.

        Show
        yip.ng added a comment - Fixed by adding code to not emit the newline. Test webapp attached.
        Hide
        Adnan Durrani added a comment -

        The following posting says that if there is a new line character between two img elements, the browsers renders a space between them.

        http://www.icefaces.org/JForum/posts/list/5397.page

        Show
        Adnan Durrani added a comment - The following posting says that if there is a new line character between two img elements, the browsers renders a space between them. http://www.icefaces.org/JForum/posts/list/5397.page
        Hide
        yip.ng added a comment -

        Updated test web app. with testing of <img> tag included.

        Show
        yip.ng added a comment - Updated test web app. with testing of <img> tag included.
        Hide
        yip.ng added a comment -

        Included <img> tag in the list of tags not to output newline.
        Also merged and re-factored methods checking node types and node names.

        Show
        yip.ng added a comment - Included <img> tag in the list of tags not to output newline. Also merged and re-factored methods checking node types and node names.

          People

          • Assignee:
            Unassigned
            Reporter:
            dukehoops
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: