Add a couple of usability and aesthetic changes, fix the half-broken chanlist functionality, add a couple of missing ESCAPED_CHANNEL vars

Add missing ESCAPED_CHANNEL
Changed EMAIL_TYPE to FORCE_EMAIL. Now makes more sense
Make it more obvious when the 'email' field is optional
Add a 'Retry' button to go back to the registration page
Fixed broken chanlist functionality. Before even if the user clicked on the channels nothing happened, now it goes straight to the 'set' page (at least something happens)
Small fix, disable the 'Save' button if user has no SET powers
Add another missing ESCAPED_CHANNEL
Add a more clear definition of what hostmasks are allowed in that field, either user@host or nick\!user@host
Fix wrongly placed var
Changed some aesthetics

6 files changed, 21 insertions, 10 deletions

diff --git a/modules/extra/webcpanel/templates/default/chanserv/access.html b/modules/extra/webcpanel/templates/default/chanserv/access.html
index 09bc26d8d..c50434f8a 100644
--- a/modules/extra/webcpanel/templates/default/chanserv/access.html
+++ b/modules/extra/webcpanel/templates/default/chanserv/access.html
@@ -46,7 +46,7 @@
 					<div class="form-group">
 						<label class="control-label col-lg-2" for="mask">Mask:</label>
 						<div class="col-lg-6">
-							<input class="form-control" type="text" name="mask" id="mask" placeholder="Valid HostMask">
+							<input class="form-control" type="text" name="mask" id="mask" placeholder="Mask must be in the form nick!user@host">
 						</div>
 					</div>
 					<div class="form-group">
diff --git a/modules/extra/webcpanel/templates/default/chanserv/akick.html b/modules/extra/webcpanel/templates/default/chanserv/akick.html
index 75cf894e2..ddfea546c 100644
--- a/modules/extra/webcpanel/templates/default/chanserv/akick.html
+++ b/modules/extra/webcpanel/templates/default/chanserv/akick.html
@@ -41,7 +41,7 @@
 					<div class="form-group">
 						<label class="control-label col-lg-2" for="mask">Mask:</label>
 						<div class="col-lg-6">
-							<input class="form-control" type="text" name="mask" id="mask" placeholder="Valid HostMask">
+							<input class="form-control" type="text" name="mask" id="mask" placeholder="Mask must be in the form nick!user@host">
 						</div>
 					</div>
 					<div class="form-group">
diff --git a/modules/extra/webcpanel/templates/default/chanserv/chanlist.html b/modules/extra/webcpanel/templates/default/chanserv/chanlist.html
index 4d386f6b0..8f00f02ac 100644
--- a/modules/extra/webcpanel/templates/default/chanserv/chanlist.html
+++ b/modules/extra/webcpanel/templates/default/chanserv/chanlist.html
@@ -1,13 +1,17 @@
-<!--- <div class="panel-heading">Channels you have access in</div> -->
+<div class="panel-heading">Channels you have access in</div>
 <div class="panel-body">
 	{IF EXISTS CHANNEL_NAMES}
 		<div>
 		<h3 style="margin: -12px 0 20px 0;"><small>Choose a channel to access it's Settings, Access or Akick pages.</small></h3>
 		{FOR CH,ECH IN CHANNEL_NAMES,ESCAPED_CHANNEL_NAMES}
-			<td><a href="{PAGE_NAME}?channel={ECH}" class="btn btn-sm btn-primary {IF EQ ECH ESCAPED_CHANNEL}disabled{END IF}">{CH}</a></td>
+			{IF EQ PAGE_NAME /chanserv/info}
+			<a href="set?channel={ECH}" class="btn btn-sm btn-primary {IF EQ ECH ESCAPED_CHANNEL}disabled{END IF}" style="margin-bottom: 4px;">{CH}</a>
+			{ELSE}
+			<a href="{PAGE_NAME}?channel={ECH}" class="btn btn-sm btn-primary {IF EQ ECH ESCAPED_CHANNEL}disabled{END IF}" style="margin-bottom: 4px;">{CH}</a>
+			{END IF}
 		{END FOR}
 		</div>
 	{ELSE}
 		<em>You don't have access in any channels.</em><br>
 	{END IF}
-</div><br>
+</div>
diff --git a/modules/extra/webcpanel/templates/default/chanserv/drop.html b/modules/extra/webcpanel/templates/default/chanserv/drop.html
index 23cb42a1e..c28eb051a 100644
--- a/modules/extra/webcpanel/templates/default/chanserv/drop.html
+++ b/modules/extra/webcpanel/templates/default/chanserv/drop.html
@@ -15,13 +15,13 @@
 						<input type="hidden" value="{CHANNEL_DROP}" name="channel">
 						<input type="hidden" value="yes" name="drop">
 						<div class="form-group">
-							<label class="control-label col-lg-3" for="confChan">Confirm channel name:</label>
+							<label class="control-label col-lg-4" for="confChan">Confirm channel name:</label>
 							<div class="col-lg-4">
 								<input class="form-control" type="text" name="confChan" id="confChan" placeholder="This cannot be undone!">
 							</div>
 						</div>
 						<div class="form-group">
-							<div class="col-lg-offset-3 col-lg-4">
+							<div class="col-lg-offset-4 col-lg-4">
 								<button type="submit" class="btn btn-danger">Drop</button>
 							</div>
 						</div>
diff --git a/modules/extra/webcpanel/templates/default/chanserv/modes.html b/modules/extra/webcpanel/templates/default/chanserv/modes.html
index 861bb0985..3c7fe7ab3 100644
--- a/modules/extra/webcpanel/templates/default/chanserv/modes.html
+++ b/modules/extra/webcpanel/templates/default/chanserv/modes.html
@@ -42,7 +42,7 @@
 					<div class="form-group">
 						<label class="control-label col-lg-2" for="mask">Mask:</label>
 						<div class="col-lg-6">
-							<input class="form-control" type="text" name="mask" id="mask" placeholder="Valid HostMask">
+							<input class="form-control" type="text" name="mask" id="mask" placeholder="Mask must be in the form nick!user@host">
 						</div>
 					</div>
 					<div class="form-group">
diff --git a/modules/extra/webcpanel/templates/default/chanserv/set.html b/modules/extra/webcpanel/templates/default/chanserv/set.html
index 0a26e3d74..050c37f69 100644
--- a/modules/extra/webcpanel/templates/default/chanserv/set.html
+++ b/modules/extra/webcpanel/templates/default/chanserv/set.html
@@ -9,7 +9,9 @@
 			{END FOR}
 
 			{IF EXISTS OKAY}
-			{IF EXISTS CAN_SET}<form method="post" action="/chanserv/set?channel={CHANNEL_ESCAPED}">{END IF}
+				{IF EXISTS CAN_SET}
+				<form method="post" action="/chanserv/set?channel={CHANNEL_ESCAPED}">
+				{END IF}
 				<table id="tableInfo" class="table table-hover">
 					<tr>
 						<td>Channel Name</td>
@@ -74,13 +76,18 @@
 						<td>Topic Lock</td>
 						<td><input type="checkbox" name="topiclock" value="on" {IF EXISTS TOPICLOCK}checked{END IF}></td>
 					</tr>
+					{IF EXISTS CAN_SET}
 					<tr>
 						<td></td>
 						<td><button type="submit" class="btn btn-primary">Save</button></td>
 					</tr>
 					{END IF}
+					{END IF}
 				</table>
-			{IF EXISTS CAN_SET}</form>{END IF}
+
+				{IF EXISTS CAN_SET}
+				</form>
+				{END IF}
 			{END IF}
 		</div>
 {INCLUDE footer.html}