TOU content not rendering under certain conditions

  • Resolved yezzz

    (@yezzz)


    4 years ago

    Did some investigation myself and found the cause.

    First, found typo </p>p> in wpg-form.php near end of line 1095.

    This didn’t fix it. Then thought it was related to thickbox vs wpautop, but that wasn’t it either. Note that this page says you need to wrap it. You don’t need p tags. You actually don’t need any wrapper (I checked), but I recommend using a div.

    As I was testing on a newer wordpress version, I then thought it was the age-old thickbox jQuery code (eg. deprecated .bind instead of .on) and newer version of jquery finally catching up with it, but that wasn’t it either.

    I was testing the ecard form page without query parameters (eg ?image=…) and finally found this code in thickbox var queryString = url.replace(/^[^\?]+\??/,''); returning an empty string (url is taken from the a.thickbox href, including the page url). queryString is then parsed to find inlineId pointing to the element with content…

    I’m posting this anyway, for future reference, and for you to fix the html.

    • This topic was modified 4 years ago by yezzz.
    • This topic was modified 4 years ago by yezzz.
Viewing 2 replies - 1 through 2 (of 2 total)
  • Plugin Author tuxlog

    (@tuxlog)

    I am not sure I got what you want to describe but just fixed the html for the next release. Is there any problem left?

    Thread Starter yezzz

    (@yezzz)

    No, just be advised thickbox needs url parameters for that particular use case. Also be prepared that some day a wordpress jquery update will render thickbox useless.

Viewing 2 replies - 1 through 2 (of 2 total)
  • The topic ‘TOU content not rendering under certain conditions’ is closed to new replies.