Extra spacing/padding

  • Resolved typeless

    (@jbalyo)


    8 years ago

    Hello-

    I found a minor bug which I have already found a fix for, but wanted to let you know about it so you can maybe incorporate it into the next version. If this were GitHub I’d just make a pull request, but alas.

    Basically, when using the shortcode in a page, it adds extra spacing above the form which shouldn’t be there – about 300 pixels or so between the form and its nearest content to the north.

    The issue is that WordPress seems to interpret newline characters (\n) in this area as line breaks. In one of the plugin files (/lib/esu-front-end-class.php), newline characters are used liberally – I assume this is to make the resulting code look cleaner… unfortunately, it is having unintended consequences.

    To fix it, I just commented out all of the line breaks from /easy-signup/lib/esu-front-end-class.php. These lines in particular:

    $esu_return .= “\n”; // line break

    Thanks, hope this helps.

    https://wordpress.org/plugins/easy-sign-up/

Viewing 3 replies - 1 through 3 (of 3 total)
  • Plugin Author Greenweb

    (@greenweb)

    8 years ago

    I use the plugin and haven’t run into this you can see it in action here: http://www.beforesite.com/support/

    I wonder if the issue is specific to your server set up. Do you mind posting what you’re running and I’ll recreate it, Thanks!

    Thread Starter typeless

    (@jbalyo)

    8 years ago

    This particular WordPress install is hosted on a shared server running Linux and PHP 5.4.45, WordPress 4.4.2… not sure what else you need?

    The newline characters don’t seem to serve any actual purpose other than formatting source code – not as important as proper formatting on the front end imo.

    Thread Starter typeless

    (@jbalyo)

    8 years ago

    I should note that the newline characters I’m speaking of are between hidden form elements – there really does not need to be spacing added between invisible elements.

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘Extra spacing/padding’ is closed to new replies.