Tapestry
  1. Tapestry
  2. TAPESTRY-2320

Problems with validation popups on Internet Explorer

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 5.0.11
    • Fix Version/s: 5.0.12
    • Component/s: tapestry-core
    • Labels:
      None
    • Environment:
      ie 7, windows xp

      Description

      There's a number of problems. Validation errors invoke stop() on the DOM event, which works in Firefox but not IE.

      In addition, some of the validation is not showing up correctly; the popups are being truncated unnecessarily.

        Activity

        Hide
        Howard M. Lewis Ship added a comment -

        So calling domevent.stop() should work on IE (it's documented that way, works on FF) but doesn't. Switching to Event.stop(domevent) seems to fix things.

        Show
        Howard M. Lewis Ship added a comment - So calling domevent.stop() should work on IE (it's documented that way, works on FF) but doesn't. Switching to Event.stop(domevent) seems to fix things.
        Hide
        Howard M. Lewis Ship added a comment -

        Experimenting with IE Developer Toolbar (a dumbed-down version of Firebug) shows something interesting; in a two column table driven layout, the left side error popups work fine, but the right side ones are wierd; the inner <span> is layout out the text as a vertical column, wrapping at all whitespace. Strange.

        Show
        Howard M. Lewis Ship added a comment - Experimenting with IE Developer Toolbar (a dumbed-down version of Firebug) shows something interesting; in a two column table driven layout, the left side error popups work fine, but the right side ones are wierd; the inner <span> is layout out the text as a vertical column, wrapping at all whitespace. Strange.
        Hide
        Howard M. Lewis Ship added a comment -

        Looks like the layout issues can be addressed by adding the error popup DIVs to the bottom: of the <body> element, rather than after: the field. Seems to work fine in FF and IE 7.

        Show
        Howard M. Lewis Ship added a comment - Looks like the layout issues can be addressed by adding the error popup DIVs to the bottom: of the <body> element, rather than after: the field. Seems to work fine in FF and IE 7.
        Hide
        Andreas Andreou added a comment -

        I've seen yahoo adding its hint popups at the end of the document too... was wondering why!

        Show
        Andreas Andreou added a comment - I've seen yahoo adding its hint popups at the end of the document too... was wondering why!
        Hide
        Christophe Cordenier added a comment -

        I have tested the Event.stop() workaround, but it still doesn't work with IE 6.

        We have also added something like this to register the "onsubmit" event of the form in the linkZone" method :

        Event.observe(element, "submit", handler);

        With this fix it works

        I Hope this can help

        Show
        Christophe Cordenier added a comment - I have tested the Event.stop() workaround, but it still doesn't work with IE 6. We have also added something like this to register the "onsubmit" event of the form in the linkZone" method : Event.observe(element, "submit", handler); With this fix it works I Hope this can help
        Hide
        Christophe Cordenier added a comment -

        Excuse me, I omit to say that in my case the problem was that the 'event' parameter passed to the 'handler' method was 'undefined'

        Show
        Christophe Cordenier added a comment - Excuse me, I omit to say that in my case the problem was that the 'event' parameter passed to the 'handler' method was 'undefined'

          People

          • Assignee:
            Howard M. Lewis Ship
            Reporter:
            Howard M. Lewis Ship
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development