Show loading spinner until share settings are fully loaded

The inputs of the sharing settings are generated in a template in the
server, but the listeners to react to changes in the elements and save
the values in the server are loaded in the client once the DOM finishes
loading. If the DOM takes long to load the user can start to interact
with the settings before the listeners were set up and, therefore, the
changes would not be saved in the server. However, as the inputs are
modified the user would think that the changes were already saved.

To address that now when the sharing settings are open a loading spinner
is shown instead of the contents of the sharing settings, and only once
the listeners to save the changes were set up the spinner is removed and
the contents shown.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

apps/settings/css/settings.scss

@@ -1129,6 +1129,10 @@ table.grid td.date {
 }
 
 #shareAPI {
+	&.loading > div {
+		display: none;
+	}
+
 	p {
 		padding-bottom: 0.8em;
 	}

apps/settings/src/admin.js

@@ -286,4 +286,6 @@ window.addEventListener('DOMContentLoaded', () => {
 	if (document.getElementById('security-warning') !== null) {
 		setupChecks()
 	}
+
+	$('#shareAPI').removeClass('loading')
 })

apps/settings/templates/settings/admin/sharing.php

@@ -27,7 +27,7 @@
 
 ?>
 
-<div class="section" id="shareAPI">
+<div class="section loading" id="shareAPI">
 	<h2><?php p($l->t('Sharing'));?></h2>
 	<?php if ($_['sharingAppEnabled'] === false) { ?>
 		<p class="warning"><?php p($l->t('You need to enable the File sharing App.')); ?></p>