We launched new forums in March 2019—join us there. In a hurry for help with your website? Get Help Now!
    • 16702 ☆ A M B ☆
    • 536 Posts
    hello,

    this add-on is great but i think <table> is...old - semantics like DIV smiley))

    bets is:
    <h2>'.$lang_mailinglist.'</h2>
    		
    	<form id="enSubscribe" action="" onsubmit="return validate_form(this);" method="post">
    	<div><label for="enFirstname">'.$lang_firstname.'</label>
    		<input type="text" name="firstname" id="enFirstname" size="50" maxlength="50" /></div>
    	<div><label for="enLastname">'.$lang_lastname.'</label>
    			<input type="text" name="lastname" id="enLastname" size="50" maxlength="50" /></div>
    	<div>		
    		<label for="enEmail">'.$lang_email.'</label>
    		<input type="text" name="email" id="enEmail" size="50" maxlength="50" />
    		<input type="hidden" name="op" value="set" /></div>
    	<div><input type="radio" name="option" value="subscribe" checked="checked" />'.$lang_subscribe.'
    		<input type="radio" name="option" value="unsubscribe" />'.$lang_unsubscribe.'</div>
    	<div><input type="submit" value="'.$lang_submit.'" /></div>
    	</form>';
    
    http://modxcms.com/forums/index.php/topic,17281.msg126710.html#msg126710
    


    or somthing like this...forgot

    GREAT WORK, thank you AGAIN!

    Kristalin
      palma non sine pulvere
      • 27708 MODX Staff
      • 2,502 Posts
      Kristalin

      If you really want to go crazy you can drop the divs altogether and use fieldset and wrap the inputs in labels like so:
      <h2>'.$lang_mailinglist.'</h2>
      		
      	<form id="enSubscribe" action="" onsubmit="return validate_form(this);" method="post">
      		<fieldset>
      			<legend>Sign Up</legend> 
      			<label for="enFirstname">'.$lang_firstname.' 
      				<input type="text" name="firstname" id="enFirstname" size="50" maxlength="50" />
      			</label> <label for="enLastname">'.$lang_lastname.' 
      				<input type="text" name="lastname" id="enLastname" size="50" maxlength="50" />
      			</label> 
      			<label for="enEmail">'.$lang_email.' 
      				<input type="text" name="email" id="enEmail" size="50" maxlength="50" />
      			</label>
      			<input type="hidden" name="op" value="set" />
      			<input type="radio" name="option" value="subscribe" checked="checked" />'.$lang_subscribe.' 
      			<input type="radio" name="option" value="unsubscribe" />'.$lang_unsubscribe.' 
      			<input type="submit" value="'.$lang_submit.'" />
      		</fieldset>
      	</form>';


      If you are not styling this with CSS you will need to add a
      after the label text but if you are nitpicking you could add style="display:block" to the inputs within the labels.

        Author of zero books. Formerly of many strange things. Pairs well with meats. Conversations are magical experiences. He's dangerous around code but a markup magician. Blog ✦ Twitter ✦ LinkedIn ✦ GitHub
        • 16702 ☆ A M B ☆
        • 536 Posts
        smashingred,
        yes is possibly but no problem just with div`s...is valid.

        i have another advice:
        in the snippet line ~64 where is the <script type="text/javascript"> after this put <![CDATA[
        and before line ~86 whre is the </script>’; put the ]]> for example:
        	<script type="text/javascript">
        <![CDATA[
        	function validate_email(field,alerttxt)
        	{
        	with (field)
        	{
        	apos=value.indexOf("@")
        	dotpos=value.lastIndexOf(".")
        	if (apos<1||dotpos-apos<2) 
        	  {alert(alerttxt);return false}
        	else {return true}
        	}
        	}
        
        	function validate_form(thisform)
        	{
        	with (thisform)
        	{
        	if (validate_email(email,"'.$lang_notvalidemail.'")==false)
        	  {email.focus();return false}
        	}
        	}
        ]]>
        	</script>';


        this trick make this JS valid smiley)))

        Regards

        Kristalin
          palma non sine pulvere
          • 26931
          • 2,314 Posts
          Hi there,

          i’m using Easy Newsletter 0.3 and get one sytem event error i don’t know to get rid of. Easy Newsletter is working without problems though...so the error i get is: "Use of undefined constant IsQmail - assumed ’IsQmail’", i’m using smtp btw., does anybody know a solution to this?

          the other error i got was "Undefined variable: lang_subscriber_action" by adding "$lang_subscriber_action = "";" i solved the problem, but can somebody tell me what should be written in the brackets?

          thank you & cheers, j
            • 17657
            • 6 Posts
            hi,

            is there a possibility to to categorize (or group) subscribers within easy newsletter? e.g. when they sign up, they can choose one (maybe more) out of several groups. and when the publisher wants to send a newsletter, he could select, which group(s) would get the mails (one specific, several, or even all groups).

            greetings,
            lukas
              • 4310
              • 2,310 Posts
              Not without some custom coding.
              It doesn’t do groups by default, either for signing up or sending out.
              sad
                • 10076
                • 1,024 Posts
                I keep getting:

                Undefined variable: mailmethod

                It is written to use SMTP, but why not just the PHP mail. Seems to work fine with eform etc?
                  • 1450
                  • 3 Posts
                  Hello
                  I will have to run two different Newsletters on my page. What would be the best way of doing this with easy newsletter? Can I install the easy newsletter with some modification two times to have two separate newsletter systems running on my webpage?

                  Thank you in advance for some hint!
                    • 29158
                    • 7 Posts
                    Hello there,

                    I have installed easy newsletter and it worked fine, however I have changed the language in configuration and now I get these errors:

                    Warning: include(../assets/modules/easynewsletter/languages/.php) [function.include]: failed to open stream: No such file or directory in /home/xxxxx/public_html/nl/manager/processors/execute_module.processor.php(114) : eval()’d code on line 75

                    Warning: include() [function.include]: Failed opening ’../assets/modules/easynewsletter/languages/.php’ for inclusion (include_path=’/home/xxxxx/public_html/nl/manager/includes/’) in /home/xxxxx/public_html/nl/manager/processors/execute_module.processor.php(114) : eval()’d code on line 75

                    I have uninstalled the module and installed again, but these errors remain the same.

                    Who can help me with this problem?

                    Thanks in advance for your help.

                    Pieter
                      • 10076
                      • 1,024 Posts
                      Hi Pieter,

                      ran into almost the same issue yesterday, here’s my solution:

                      http://modxcms.com/forums/index.php/topic,33080.msg206647.html#msg206647

                      Let me know if that works