Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.1.9
    • Component/s: Html Editor
    • Labels:
      None
    • Environment:
      Windows OS

      Description

      The <t:inputHtml> field is initially blank. First I perform a submit on a page with required data, then the error messages are displayed.
      Then I click a <t:commandLink> to perform a lookup and return return to the page with the inputHtml, #000000 text is in the inputHtml field.

        Activity

        Hide
        Paul Rivera added a comment -

        I was able to replicate your problem and do some initial investigation. I think the problem is in the kupueditor.js.
        Checking the javascript function _serializeOutputToString(), the content generated is:

        <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
        <html xmlns="http://www.w3.org/1999/xhtml"><head><title></title></head><body>#000000</body></html>

        the content "<body>#000000</body>" is generated by:
        Sarissa.serialize(transform.getElementsByTagName("body")[0])

        This then gets decoded by the InputHtmlRenderer and saved in your backing bean.

        I'll look more into this

        Show
        Paul Rivera added a comment - I was able to replicate your problem and do some initial investigation. I think the problem is in the kupueditor.js. Checking the javascript function _serializeOutputToString(), the content generated is: <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd"> <html xmlns="http://www.w3.org/1999/xhtml"><head><title></title></head><body>#000000</body></html> the content "<body>#000000</body>" is generated by: Sarissa.serialize(transform.getElementsByTagName("body") [0] ) This then gets decoded by the InputHtmlRenderer and saved in your backing bean. I'll look more into this
        Hide
        Paul Rivera added a comment -

        After tracing through piles of kupu javascript code, I finally found the root cause and a fix.

        The content #000000 that you are getting is actually the font color of your input.

        In kupucontentfilters.js, it confuses the attribute 'text' with 'textContent'. The attribute 'text' refers to the font-color of your text while 'textContent' is your actual text input.

        I can probably submit this fix to the kupu developers as well.

        This patch has been tested with both firefox and internet explorer and works well with both.

        Show
        Paul Rivera added a comment - After tracing through piles of kupu javascript code, I finally found the root cause and a fix. The content #000000 that you are getting is actually the font color of your input. In kupucontentfilters.js, it confuses the attribute 'text' with 'textContent'. The attribute 'text' refers to the font-color of your text while 'textContent' is your actual text input. I can probably submit this fix to the kupu developers as well. This patch has been tested with both firefox and internet explorer and works well with both.
        Hide
        Paul Rivera added a comment -

        kupu's issue tracker (http://codespeak.net/issues/kupu/) has been closed due to spamming.

        I've already submitted the patch to their dev mailing list (kupu-dev@codespeak.net) last sept 18. No replies so far. Their dev mailing list seems to be inactive.

        Their last release, kupu-1.3.5, was dated 02/03/06. It will probably take longer if we wait for them to fix this.

        Show
        Paul Rivera added a comment - kupu's issue tracker ( http://codespeak.net/issues/kupu/ ) has been closed due to spamming. I've already submitted the patch to their dev mailing list (kupu-dev@codespeak.net) last sept 18. No replies so far. Their dev mailing list seems to be inactive. Their last release, kupu-1.3.5, was dated 02/03/06. It will probably take longer if we wait for them to fix this.
        Hide
        Leonardo Uribe added a comment -

        Thanks to Paul Rivera for provide this patch

        Show
        Leonardo Uribe added a comment - Thanks to Paul Rivera for provide this patch

          People

          • Assignee:
            Leonardo Uribe
            Reporter:
            Tony
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development