Validation errors from generated HTML markup

I have a couple of validation errors (and general markup cleanups) that I would like to enter into the plugin.

in e-newsletter.php:

Line 1536, the language attribute does not exist in HTML any longer and should be omitted:

<script language="JavaScript">

New:

<script type="text/javascript">

Line 1574, form elements need an action, while Wordpress manages to accept this error an action URL should be specified:

<form action="" method="post" name="subscribes_form" id="subscribes_form">

Lines 1579-1580, the label element should have a for attribute that references the id attribute of the corresponding input:

<label><?php _e( 'Your Email:', 'email-newsletter' ) ?></label>
                <input type="text" name="e_newsletter_email" id="e_newsletter_email" />

New:

<label for="e_newsletter_email"><?php _e( 'Your Email:', 'email-newsletter' ) ?></label>
                <input type="text" name="e_newsletter_email" id="e_newsletter_email" />

Lines 1586-1587, same as above:

<label><?php _e( 'Your Name:', 'email-newsletter' ) ?></label>
                    <input type="text" name="e_newsletter_name" id="e_newsletter_name" />

New:

<label for="e_newsletter_name"><?php _e( 'Your Name:', 'email-newsletter' ) ?></label>
                    <input type="text" name="e_newsletter_name" id="e_newsletter_name" />

Lines 1600-1603, same as above:

<label>
                                    <input type="checkbox" name="e_newsletter_groups_id[]" value="<?php echo $group['group_id'];?>" />
                                    <?php echo $group['group_name'];?>
                                </label>

New:

<input type="checkbox" name="e_newsletter_groups_id[]" value="<?php echo $group['group_id'];?>" id="e_newsletter_groups_id_<?php echo $group['group_id'];?>"/>
                                 <label for="e_newsletter_groups_id_<?php echo $group['group_id'];?>">
                                   <?php echo $group['group_name'];?>
                                </label>

Line 1593 has a label element with no corresponding input or form element that I can find, semantically this should probably be a heading.

<label><?php _e( 'Subscribe to:', 'email-newsletter' ) ?></label>

New:

<h3><?php _e( 'Subscribe to:', 'email-newsletter' ) ?></h3>

Line 1624 has a label element that has no corresponding input or form element, should probably be a heading:

<label><?php _e( 'Subscribe to:', 'email-newsletter' ) ?></label>

New:

<h3><?php _e( 'Subscribe to:', 'email-newsletter' ) ?></h3>

Line 1631: checked is a boolean attribute:

$checked = 'checked="checked"';

New:

$checked = 'checked';

Lines 1634-1636, labels again:

<label>
                                    <input type="checkbox" name="e_newsletter_groups_id[]" value="<?php echo $group['group_id'];?>" <?php echo $checked;?> />
                                    <?php echo $group['group_name'];?>
                                </label>

New:

<input type="checkbox" name="e_newsletter_groups_id[]" value="<?php echo $group['group_id'];?>" <?php echo $checked;?> id="e_newsletter_groups_id<?php echo $group['group_id'];?>"/>
<label for="e_newsletter_groups_id<?php echo $group['group_id'];?>">
                                    <?php echo $group['group_name'];?>
                                </label>

Lines 1687-1690, 1693-1696 and 1699-1702 also follow this pattern.

And a question for the developer: Have you considered splitting the javascript into a separate file that is included via wp_register_script rather than printing it directly into the page?