From 5227a48a66614cf1d17bdc7105bbc40c71ebb2d7 Mon Sep 17 00:00:00 2001
From: Cooper <cfellows@champlain.edu>
Date: Wed, 25 Jan 2017 16:05:54 -0500
Subject: [PATCH 01/11] Attempt to add unique item ID to input

Mostly testing this as a proof of concept, each input in our forms
should have a unique ID, so that the form labels can be linked to them.
---
 templates/partials/result_item/table_header.php | 2 +-
 templates/partials/results_items.php            | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/templates/partials/result_item/table_header.php b/templates/partials/result_item/table_header.php
index ed8e2191a..6db7b0b15 100644
--- a/templates/partials/result_item/table_header.php
+++ b/templates/partials/result_item/table_header.php
@@ -21,7 +21,7 @@
 <hr>
 <p>Select which part of the table to convert to a header</p>
 <div class="input-group">
-	<select class="form-control" name="newcontent">
+	<select class="form-control" name="newcontent" id="<?= $this->e($item_id); ?>">
 		<option value="row">The first row</option>
 		<option value="col">The first column</option>
 		<option value="both">Both the first row and column</option>
diff --git a/templates/partials/results_items.php b/templates/partials/results_items.php
index bf4eab095..df0f8f4b5 100644
--- a/templates/partials/results_items.php
+++ b/templates/partials/results_items.php
@@ -145,7 +145,7 @@
 											}
 
 											if ( ! empty($result_template)) {
-												echo($this->fetch("partials/result_item/{$result_template}", ['group_item' => $group_item]));
+												echo($this->fetch("partials/result_item/{$result_template}", ['group_item' => $group_item, 'item_id' => $li_id]));
 											}
 										?>
 									</form>

From c45e72ff56b44416f4d64c4a267a89fce9345d0e Mon Sep 17 00:00:00 2001
From: Cooper <cfellows@champlain.edu>
Date: Wed, 25 Jan 2017 16:16:58 -0500
Subject: [PATCH 02/11] Add label that references our new dynamic ID

---
 templates/partials/result_item/table_header.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/templates/partials/result_item/table_header.php b/templates/partials/result_item/table_header.php
index 6db7b0b15..10c138de4 100644
--- a/templates/partials/result_item/table_header.php
+++ b/templates/partials/result_item/table_header.php
@@ -19,7 +19,7 @@
 */
 ?>
 <hr>
-<p>Select which part of the table to convert to a header</p>
+<label for="<?= $this->e($item_id); ?>">Select which part of the table to convert to a header</label>
 <div class="input-group">
 	<select class="form-control" name="newcontent" id="<?= $this->e($item_id); ?>">
 		<option value="row">The first row</option>

From 3ce43d9ced64322a1dead9c9499dbc18e82d0905 Mon Sep 17 00:00:00 2001
From: Cooper <cfellows@champlain.edu>
Date: Wed, 25 Jan 2017 16:28:03 -0500
Subject: [PATCH 03/11] Add unique ID's to contrast fixes BG and FG inputs

Need to handle the Color Pallete System, that could use some love from
an A11Y standpoint
---
 templates/partials/result_item/contrast.php | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/templates/partials/result_item/contrast.php b/templates/partials/result_item/contrast.php
index 101a4de3d..4d47f3d01 100644
--- a/templates/partials/result_item/contrast.php
+++ b/templates/partials/result_item/contrast.php
@@ -26,12 +26,12 @@
 	<div class="left">
 		<div class="form-group no-margin margin-bottom">
 			<?php if ( isset($group_item->back_color) ): ?>
-				<label for="newcontent[1]">Replace Background Color <?= $group_item->back_color; ?></label>
-				<input class="color {hash:true,caps:false} form-control back-color" type="text" name="newcontent[1]" value="<?= $group_item->back_color; ?>" placeholder="Replacement for Background Color <?= $group_item->back_color; ?>">
+				<label for="<?= $this->e($item_id); ?>-bgcolor">Replace Background Color <?= $group_item->back_color; ?></label>
+				<input class="color {hash:true,caps:false} form-control back-color" type="text" name="newcontent[1]" value="<?= $group_item->back_color; ?>" placeholder="Replacement for Background Color <?= $group_item->back_color; ?>" id="<?= $this->e($item_id);-bgcolor?>">
 			<?php endif; ?>
 
-			<label for="newcontent[0]">Replace Foreground Color <?= $group_item->fore_color; ?></label>&nbsp;<span class="contrast-invalid hidden red"><span class="glyphicon glyphicon-remove"></span>&nbsp;Ratio Invalid (<span class="contrast-ratio"></span>:1)</span>
-			<input class="color {hash:true,caps:false} form-control fore-color" type="text" name="newcontent[0]" value="<?= $group_item->fore_color; ?>" placeholder="Replacement for Foreground Color <?= $group_item->fore_color; ?>">
+			<label for="<?= $this->e($item_id); ?>-fgcolor">Replace Foreground Color <?= $group_item->fore_color; ?></label>&nbsp;<span class="contrast-invalid hidden red"><span class="glyphicon glyphicon-remove"></span>&nbsp;Ratio Invalid (<span class="contrast-ratio"></span>:1)</span>
+			<input class="color {hash:true,caps:false} form-control fore-color" type="text" name="newcontent[0]" value="<?= $group_item->fore_color; ?>" placeholder="Replacement for Foreground Color <?= $group_item->fore_color; ?>" id="<?= $this->e($item_id); ?>-fgcolor">
 			<label><input name="add-bold" type="checkbox" value="bold" />&nbsp;Make this text bold</label>&nbsp;<label><input name="add-italic" type="checkbox" value="italic" />&nbsp;Make this text <span style="font-style: italics;">italicized</span></label><br />
 			<input type="text" name="threshold" class="threshold hidden" value="<?= $group_item->text_type; ?>">
 		</div>

From 628c6e52bb5ad96de13af8f3e6d94230442ee25e Mon Sep 17 00:00:00 2001
From: Cooper <cfellows@champlain.edu>
Date: Wed, 25 Jan 2017 16:47:28 -0500
Subject: [PATCH 04/11] Add label for missing header text error

---
 templates/partials/result_item/header_text.php | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/templates/partials/result_item/header_text.php b/templates/partials/result_item/header_text.php
index 15fb7ea4a..bcf8408b8 100644
--- a/templates/partials/result_item/header_text.php
+++ b/templates/partials/result_item/header_text.php
@@ -19,7 +19,8 @@
 */
 ?>
 <div class="form-group no-margin margin-bottom">
-	<input class="{hash:true,caps:false} form-control" type="text" name="newcontent" placeholder="New heading text">
+	<label for="<?= $this->e($item_id); ?>">Enter New Text For This Heading</label>
+	<input class="{hash:true,caps:false} form-control" type="text" name="newcontent" placeholder="New heading text" id="<?= $this->e($item_id); ?>">
 	<label><input class="remove-heading" type="checkbox" />&nbsp;Delete this Header completely instead</label><br />
 	<button class="submit-content inactive btn btn-default" type="submit">Submit</button>
 </div>

From 2b752b1f8407a1b3861d24358dd17a368b3a9451 Mon Sep 17 00:00:00 2001
From: Cooper <cfellows@champlain.edu>
Date: Wed, 25 Jan 2017 16:56:37 -0500
Subject: [PATCH 05/11] Adjust layout to hide the label when not required

We are adding the labels for screen readers, but in many cases, the
placeholder text is good enough for operation when the labels are not
needed for sighted individuals.
---
 templates/partials/result_item/header_text.php  | 2 +-
 templates/partials/result_item/table_header.php | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/templates/partials/result_item/header_text.php b/templates/partials/result_item/header_text.php
index bcf8408b8..33611563e 100644
--- a/templates/partials/result_item/header_text.php
+++ b/templates/partials/result_item/header_text.php
@@ -19,7 +19,7 @@
 */
 ?>
 <div class="form-group no-margin margin-bottom">
-	<label for="<?= $this->e($item_id); ?>">Enter New Text For This Heading</label>
+	<label for="<?= $this->e($item_id); ?>" class="control-label sr-only">Enter New Text For This Heading</label>
 	<input class="{hash:true,caps:false} form-control" type="text" name="newcontent" placeholder="New heading text" id="<?= $this->e($item_id); ?>">
 	<label><input class="remove-heading" type="checkbox" />&nbsp;Delete this Header completely instead</label><br />
 	<button class="submit-content inactive btn btn-default" type="submit">Submit</button>
diff --git a/templates/partials/result_item/table_header.php b/templates/partials/result_item/table_header.php
index 10c138de4..87a08c8d1 100644
--- a/templates/partials/result_item/table_header.php
+++ b/templates/partials/result_item/table_header.php
@@ -19,8 +19,8 @@
 */
 ?>
 <hr>
-<label for="<?= $this->e($item_id); ?>">Select which part of the table to convert to a header</label>
 <div class="input-group">
+	<label for="<?= $this->e($item_id); ?>" class="control-label sr-only">Select which part of the table to convert to a header</label>
 	<select class="form-control" name="newcontent" id="<?= $this->e($item_id); ?>">
 		<option value="row">The first row</option>
 		<option value="col">The first column</option>

From e3d89de82ccc4823e023be631ec71d7ec689e776 Mon Sep 17 00:00:00 2001
From: Cooper <cfellows@champlain.edu>
Date: Wed, 25 Jan 2017 16:56:50 -0500
Subject: [PATCH 06/11] Add form label for image alt error

---
 templates/partials/result_item/image_alt.php | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/templates/partials/result_item/image_alt.php b/templates/partials/result_item/image_alt.php
index 79d609ceb..7ee95745a 100644
--- a/templates/partials/result_item/image_alt.php
+++ b/templates/partials/result_item/image_alt.php
@@ -19,8 +19,9 @@
 */
 ?>
 <div class="fix-alt input-group">
+	<label for="<?= $this->e($item_id); ?>" class="control-label sr-only">Provide New Alt Text For This Image</label>
 	<span class="counter">100</span>
-	<input class="form-control" type="text" name="newcontent" maxlength="100" placeholder="New alt text">
+	<input class="form-control" type="text" name="newcontent" maxlength="100" placeholder="New alt text" id="<?= $this->e($item_id); ?>">
 	<span class="input-group-btn">
 		<button class="submit-content inactive btn btn-default" type="submit">Submit</button>
 	</span>

From 7f34c0d68a6a6544dcaeed2107b858831bbfbfd2 Mon Sep 17 00:00:00 2001
From: Cooper <cfellows@champlain.edu>
Date: Wed, 25 Jan 2017 17:05:18 -0500
Subject: [PATCH 07/11] Add form labels to the link and link_text templates

Note that these appear to be duplicative templates and one can probably
be removed, but look at the results_items.php statement to be sure.
---
 templates/partials/result_item/link.php      | 3 ++-
 templates/partials/result_item/link_text.php | 3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/templates/partials/result_item/link.php b/templates/partials/result_item/link.php
index d0deab462..f019a5ba8 100644
--- a/templates/partials/result_item/link.php
+++ b/templates/partials/result_item/link.php
@@ -19,7 +19,8 @@
 */
 ?>
 <div class="form-group no-margin margin-bottom">
-	<input class="{hash:true,caps:false} form-control" type="text" name="newcontent" placeholder="New link text">
+	<label for="<?= $this->e($item_id); ?>" class="control-label sr-only">Enter New Text for This Link</label>
+	<input class="{hash:true,caps:false} form-control" type="text" name="newcontent" placeholder="New link text" id="<?= $this->e($item_id); ?>">
 	<label><input class="remove-link" type="checkbox" />Delete this Link completely instead</label><br />
 	<button class="submit-content inactive btn btn-default" type="submit">Submit</button>
 </div>
diff --git a/templates/partials/result_item/link_text.php b/templates/partials/result_item/link_text.php
index 72c3e7d21..788a55f6d 100644
--- a/templates/partials/result_item/link_text.php
+++ b/templates/partials/result_item/link_text.php
@@ -19,7 +19,8 @@
 */
 ?>
 <div class="form-group no-margin margin-bottom">
-	<input class="{hash:true,caps:false} form-control" type="text" name="newcontent" placeholder="New link text">
+	<label for="<?= $this->e($item_id); ?>" class="control-label sr-only">Enter New Text for This Link</label>
+	<input class="{hash:true,caps:false} form-control" type="text" name="newcontent" placeholder="New link text" id="<?= $this->e($item_id); ?>">
 	<label><input class="remove-link" type="checkbox" />&nbsp;Delete this Link completely instead</label><br />
 	<button class="submit-content inactive btn btn-default" type="submit">Submit</button>
 </div>

From 3cff78a5af8a9510af16ffd770c7e48fe61288a6 Mon Sep 17 00:00:00 2001
From: Cooper <cfellows@champlain.edu>
Date: Wed, 25 Jan 2017 17:07:43 -0500
Subject: [PATCH 08/11] Add form label

This template also appears to be duplicative, like the link and
link_text templates. In fact, according to the switch statement in
results_items.php, this case would never be reached, since
"aSuspiciousLinkText" would be twice before this template is ever used
(lines 103 and 120 in that file)
---
 templates/partials/result_item/suspicious_link_text.php | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/templates/partials/result_item/suspicious_link_text.php b/templates/partials/result_item/suspicious_link_text.php
index f468c97e1..a29a5cf73 100644
--- a/templates/partials/result_item/suspicious_link_text.php
+++ b/templates/partials/result_item/suspicious_link_text.php
@@ -19,7 +19,8 @@
 */
 ?>
 <div class="input-group">
-	<input class="form-control" type="text" name="newcontent" placeholder="New link description">
+	<label for="<?= $this->e($item_id); ?>" class="control-label sr-only">Enter a New Description for This Link</label>
+	<input class="form-control" type="text" name="newcontent" placeholder="New link description" id="<?= $this->e($item_id); ?>">
 	<span class="input-group-btn">
 		<button class="submit-content inactive btn btn-default" type="submit">Submit</button>
 	</span>

From 659eec403449272537439d372c09b78b28a1efd1 Mon Sep 17 00:00:00 2001
From: Cooper <cfellows@champlain.edu>
Date: Wed, 25 Jan 2017 17:11:44 -0500
Subject: [PATCH 09/11] Add form label to thead scope

This one needs a visible label, so it lives outside the form group, per:
http://getbootstrap.com/components/#input-groups-basic
---
 templates/partials/result_item/table_header_scope.php | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/templates/partials/result_item/table_header_scope.php b/templates/partials/result_item/table_header_scope.php
index 74316392e..bce5b779f 100644
--- a/templates/partials/result_item/table_header_scope.php
+++ b/templates/partials/result_item/table_header_scope.php
@@ -18,8 +18,9 @@
 *	Primary Author Contact:  Jacob Bates <jacob.bates@ucf.edu>
 */
 ?>
+<label for="<?= $this->e($item_id); ?>" class="control-label sr-only">Select which part of the table to convert to a header</label>
 <div class="input-group">
-	<select class="form-control" name="newcontent">
+	<select class="form-control" name="newcontent" id="<?= $this->e($item_id); ?>">
 		<option value="col">col</option>
 		<option value="row">row</option>
 	</select>

From b04621442ede157ac4071de1833d140ea4cfd225 Mon Sep 17 00:00:00 2001
From: Cooper <cfellows@champlain.edu>
Date: Wed, 25 Jan 2017 17:21:59 -0500
Subject: [PATCH 10/11] Ensure input ID's are actually unique

Was using the item_id passed from earlier on in the templating process,
but that same ID is put on the <li> item to enable collapsing of
results.
---
 templates/partials/result_item/header_text.php          | 4 ++--
 templates/partials/result_item/image_alt.php            | 4 ++--
 templates/partials/result_item/link.php                 | 4 ++--
 templates/partials/result_item/link_text.php            | 4 ++--
 templates/partials/result_item/suspicious_link_text.php | 2 +-
 templates/partials/result_item/table_header.php         | 4 ++--
 templates/partials/result_item/table_header_scope.php   | 2 +-
 7 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/templates/partials/result_item/header_text.php b/templates/partials/result_item/header_text.php
index 33611563e..d4501426b 100644
--- a/templates/partials/result_item/header_text.php
+++ b/templates/partials/result_item/header_text.php
@@ -19,8 +19,8 @@
 */
 ?>
 <div class="form-group no-margin margin-bottom">
-	<label for="<?= $this->e($item_id); ?>" class="control-label sr-only">Enter New Text For This Heading</label>
-	<input class="{hash:true,caps:false} form-control" type="text" name="newcontent" placeholder="New heading text" id="<?= $this->e($item_id); ?>">
+	<label for="<?= $this->e($item_id); ?>-input" class="control-label sr-only">Enter New Text For This Heading</label>
+	<input class="{hash:true,caps:false} form-control" type="text" name="newcontent" placeholder="New heading text" id="<?= $this->e($item_id); ?>-input">
 	<label><input class="remove-heading" type="checkbox" />&nbsp;Delete this Header completely instead</label><br />
 	<button class="submit-content inactive btn btn-default" type="submit">Submit</button>
 </div>
diff --git a/templates/partials/result_item/image_alt.php b/templates/partials/result_item/image_alt.php
index 7ee95745a..c99457e12 100644
--- a/templates/partials/result_item/image_alt.php
+++ b/templates/partials/result_item/image_alt.php
@@ -19,9 +19,9 @@
 */
 ?>
 <div class="fix-alt input-group">
-	<label for="<?= $this->e($item_id); ?>" class="control-label sr-only">Provide New Alt Text For This Image</label>
+	<label for="<?= $this->e($item_id); ?>-input" class="control-label sr-only">Provide New Alt Text For This Image</label>
 	<span class="counter">100</span>
-	<input class="form-control" type="text" name="newcontent" maxlength="100" placeholder="New alt text" id="<?= $this->e($item_id); ?>">
+	<input class="form-control" type="text" name="newcontent" maxlength="100" placeholder="New alt text" id="<?= $this->e($item_id); ?>-input">
 	<span class="input-group-btn">
 		<button class="submit-content inactive btn btn-default" type="submit">Submit</button>
 	</span>
diff --git a/templates/partials/result_item/link.php b/templates/partials/result_item/link.php
index f019a5ba8..d3730811a 100644
--- a/templates/partials/result_item/link.php
+++ b/templates/partials/result_item/link.php
@@ -19,8 +19,8 @@
 */
 ?>
 <div class="form-group no-margin margin-bottom">
-	<label for="<?= $this->e($item_id); ?>" class="control-label sr-only">Enter New Text for This Link</label>
-	<input class="{hash:true,caps:false} form-control" type="text" name="newcontent" placeholder="New link text" id="<?= $this->e($item_id); ?>">
+	<label for="<?= $this->e($item_id); ?>-input" class="control-label sr-only">Enter New Text for This Link</label>
+	<input class="{hash:true,caps:false} form-control" type="text" name="newcontent" placeholder="New link text" id="<?= $this->e($item_id); ?>-input">
 	<label><input class="remove-link" type="checkbox" />Delete this Link completely instead</label><br />
 	<button class="submit-content inactive btn btn-default" type="submit">Submit</button>
 </div>
diff --git a/templates/partials/result_item/link_text.php b/templates/partials/result_item/link_text.php
index 788a55f6d..4ad859dfa 100644
--- a/templates/partials/result_item/link_text.php
+++ b/templates/partials/result_item/link_text.php
@@ -19,8 +19,8 @@
 */
 ?>
 <div class="form-group no-margin margin-bottom">
-	<label for="<?= $this->e($item_id); ?>" class="control-label sr-only">Enter New Text for This Link</label>
-	<input class="{hash:true,caps:false} form-control" type="text" name="newcontent" placeholder="New link text" id="<?= $this->e($item_id); ?>">
+	<label for="<?= $this->e($item_id); ?>-input" class="control-label sr-only">Enter New Text for This Link</label>
+	<input class="{hash:true,caps:false} form-control" type="text" name="newcontent" placeholder="New link text" id="<?= $this->e($item_id); ?>-input">
 	<label><input class="remove-link" type="checkbox" />&nbsp;Delete this Link completely instead</label><br />
 	<button class="submit-content inactive btn btn-default" type="submit">Submit</button>
 </div>
diff --git a/templates/partials/result_item/suspicious_link_text.php b/templates/partials/result_item/suspicious_link_text.php
index a29a5cf73..0875ceeda 100644
--- a/templates/partials/result_item/suspicious_link_text.php
+++ b/templates/partials/result_item/suspicious_link_text.php
@@ -19,7 +19,7 @@
 */
 ?>
 <div class="input-group">
-	<label for="<?= $this->e($item_id); ?>" class="control-label sr-only">Enter a New Description for This Link</label>
+	<label for="<?= $this->e($item_id); ?>-input" class="control-label sr-only">Enter a New Description for This Link</label>
 	<input class="form-control" type="text" name="newcontent" placeholder="New link description" id="<?= $this->e($item_id); ?>">
 	<span class="input-group-btn">
 		<button class="submit-content inactive btn btn-default" type="submit">Submit</button>
diff --git a/templates/partials/result_item/table_header.php b/templates/partials/result_item/table_header.php
index 87a08c8d1..7090de6c3 100644
--- a/templates/partials/result_item/table_header.php
+++ b/templates/partials/result_item/table_header.php
@@ -20,8 +20,8 @@
 ?>
 <hr>
 <div class="input-group">
-	<label for="<?= $this->e($item_id); ?>" class="control-label sr-only">Select which part of the table to convert to a header</label>
-	<select class="form-control" name="newcontent" id="<?= $this->e($item_id); ?>">
+	<label for="<?= $this->e($item_id); ?>-input" class="control-label sr-only">Select which part of the table to convert to a header</label>
+	<select class="form-control" name="newcontent" id="<?= $this->e($item_id); ?>-input">
 		<option value="row">The first row</option>
 		<option value="col">The first column</option>
 		<option value="both">Both the first row and column</option>
diff --git a/templates/partials/result_item/table_header_scope.php b/templates/partials/result_item/table_header_scope.php
index bce5b779f..c57aaea45 100644
--- a/templates/partials/result_item/table_header_scope.php
+++ b/templates/partials/result_item/table_header_scope.php
@@ -18,7 +18,7 @@
 *	Primary Author Contact:  Jacob Bates <jacob.bates@ucf.edu>
 */
 ?>
-<label for="<?= $this->e($item_id); ?>" class="control-label sr-only">Select which part of the table to convert to a header</label>
+<label for="<?= $this->e($item_id); ?>-input" class="control-label sr-only">Select which part of the table to convert to a header</label>
 <div class="input-group">
 	<select class="form-control" name="newcontent" id="<?= $this->e($item_id); ?>">
 		<option value="col">col</option>

From e0cf516216fe3da07c8da9e4a70c7f570995633c Mon Sep 17 00:00:00 2001
From: Cooper <cfellows@champlain.edu>
Date: Wed, 25 Jan 2017 17:30:58 -0500
Subject: [PATCH 11/11] Ensure headerScope and BGcolor have unique IDs

Typo was preventing both from working as intended
---
 templates/partials/result_item/contrast.php           | 2 +-
 templates/partials/result_item/table_header_scope.php | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/templates/partials/result_item/contrast.php b/templates/partials/result_item/contrast.php
index 4d47f3d01..b0a6d8587 100644
--- a/templates/partials/result_item/contrast.php
+++ b/templates/partials/result_item/contrast.php
@@ -27,7 +27,7 @@
 		<div class="form-group no-margin margin-bottom">
 			<?php if ( isset($group_item->back_color) ): ?>
 				<label for="<?= $this->e($item_id); ?>-bgcolor">Replace Background Color <?= $group_item->back_color; ?></label>
-				<input class="color {hash:true,caps:false} form-control back-color" type="text" name="newcontent[1]" value="<?= $group_item->back_color; ?>" placeholder="Replacement for Background Color <?= $group_item->back_color; ?>" id="<?= $this->e($item_id);-bgcolor?>">
+				<input class="color {hash:true,caps:false} form-control back-color" type="text" name="newcontent[1]" value="<?= $group_item->back_color; ?>" placeholder="Replacement for Background Color <?= $group_item->back_color; ?>" id="<?= $this->e($item_id);?>-bgcolor">
 			<?php endif; ?>
 
 			<label for="<?= $this->e($item_id); ?>-fgcolor">Replace Foreground Color <?= $group_item->fore_color; ?></label>&nbsp;<span class="contrast-invalid hidden red"><span class="glyphicon glyphicon-remove"></span>&nbsp;Ratio Invalid (<span class="contrast-ratio"></span>:1)</span>
diff --git a/templates/partials/result_item/table_header_scope.php b/templates/partials/result_item/table_header_scope.php
index c57aaea45..94af79a9d 100644
--- a/templates/partials/result_item/table_header_scope.php
+++ b/templates/partials/result_item/table_header_scope.php
@@ -20,7 +20,7 @@
 ?>
 <label for="<?= $this->e($item_id); ?>-input" class="control-label sr-only">Select which part of the table to convert to a header</label>
 <div class="input-group">
-	<select class="form-control" name="newcontent" id="<?= $this->e($item_id); ?>">
+	<select class="form-control" name="newcontent" id="<?= $this->e($item_id); ?>-input">
 		<option value="col">col</option>
 		<option value="row">row</option>
 	</select>