Cleaner admin settings

Signed-off-by: Joas Schilling <coding@schilljs.com>

1 files changed, 22 insertions, 16 deletions

diff --git a/templates/admin.php b/templates/admin.php
index e9aeb12..3eecc2c 100644
--- a/templates/admin.php
+++ b/templates/admin.php
@@ -2,37 +2,43 @@
 /** @var array $_ */
 /** @var \OCP\IL10N $l */
 script('registration', 'settings');
+style('registration', 'settings');
 ?>
 <form id="registration_settings_form" class="section">
 	<h2><?php p($l->t('Registration')); ?></h2><span id="registration_settings_msg" class="msg"></span>
+
+	<h3><?php p($l->t('Registered users default group')); ?></h3>
 	<p>
-	<label for="registered_user_group"><?php p($l->t('Default group that all registered users belong')); ?></label>
-	<select id="registered_user_group" name="registered_user_group">
-		<option value="none" <?php echo $_['current'] === 'none' ? 'selected="selected"' : ''; ?>><?php p($l->t('None')); ?></option>
+		<label>
+			<select id="registered_user_group" name="registered_user_group">
+				<option value="none" <?php echo $_['current'] === 'none' ? 'selected="selected"' : ''; ?>><?php p($l->t('None')); ?></option>
 <?php
 foreach ($_['groups'] as $group) {
 	$selected = $_['current'] === $group ? 'selected="selected"' : '';
 	echo '<option value="'.$group.'" '.$selected.'>'.$group.'</option>';
 }
 ?>
-	</select>
+			</select>
+		</label>
 	</p>
+
+	<h3><?php p($l->t('Allowed email domains')); ?></h3>
 	<p>
-	<label for="allowed_domains"><?php p($l->t('Allowed mail address domains for registration')); ?></label>
-	<input type="text" id="allowed_domains" name="allowed_domains" value=<?php p($_['allowed']);?>>
+		<label>
+			<input type="text" id="allowed_domains" name="allowed_domains" value="<?php p($_['allowed']);?>" placeholder="nextcloud.com;*.example.com">
+		</label>
 	</p>
 	<p>
-	<em><?php p($l->t('Enter a semicolon-separated list of allowed domains. Example: nextcloud.com;example.com'));?></em>
+		<em><?php p($l->t('Enter a semicolon-separated list of allowed domains, * for wildcard. Example: %s', ['nextcloud.com;*.example.com']));?></em>
 	</p>
 
-	<div style="margin-top: 10px;">
-		<p>
-			<input type="checkbox" id="admin_approval_required" class="checkbox" name="admin_approval_required" <?php if ($_['approval_required'] === "yes") {
-	echo " checked";
-} ?>>
-			<label for="admin_approval_required"><?php p($l->t('Require admin approval?')); ?></label>
-		</p>
+	<h3><?php p($l->t('Admin approval')); ?></h3>
+	<p>
+		<input type="checkbox" id="admin_approval_required" class="checkbox" name="admin_approval_required" <?php if ($_['approval_required'] === "yes") {
+			echo " checked";
+		} ?>>
+		<label for="admin_approval_required"><?php p($l->t('Require admin approval')); ?></label>
+	</p>
 
-		<em><?php p($l->t('Enabling "admin approval" will prevent registrations from mobile and desktop clients to complete as the credentials can not be verified by the client until the user was enabled.'));?></em>
-	</div>
+	<em><?php p($l->t('Enabling "admin approval" will prevent registrations from mobile and desktop clients to complete as the credentials can not be verified by the client until the user was enabled.'));?></em>
 </form>