From 17952c9cf39cc60a28538f067143f740855cacaf Mon Sep 17 00:00:00 2001
From: Sunil Mohan Adapa <sunil@medhas.org>
Date: Fri, 4 Oct 2024 20:30:17 -0700
Subject: [PATCH] firstboot: Improve the setup complete page with more setups

Fixes: #888.

- Suggest all the steps that a typical user should likely take.

- Custom styling to make the page look good.

- Open the links in new windows as this page can't be reached again.

- Add a button for software updates as this can be done easily and the most
important step.

Tests:

- Trigger first setup by removing plinth.sqlite3. Notice the improved setup
complete page. Text and icons are as expected. Links work and open in a new
window. Clicking on 'Update now' button opens a page to software updates with
manual upgrade triggered.

- Mobile view looks good.

Signed-off-by: Sunil Mohan Adapa <sunil@medhas.org>
Reviewed-by: Veiko Aasa <veiko17@disroot.org>
---
 .../templates/firstboot_complete.html         | 90 +++++++++++++++----
 static/themes/default/css/main.css            | 23 +++++
 2 files changed, 94 insertions(+), 19 deletions(-)

diff --git a/plinth/modules/first_boot/templates/firstboot_complete.html b/plinth/modules/first_boot/templates/firstboot_complete.html
index d0323dd75..b931395dc 100644
--- a/plinth/modules/first_boot/templates/firstboot_complete.html
+++ b/plinth/modules/first_boot/templates/firstboot_complete.html
@@ -8,27 +8,79 @@
 
 {% block content %}
 
-  <h2>{% trans "Setup Complete!" %}</h2>
+  <h2>{% trans "Setup Complete! Next Steps:" %}</h2>
 
-  <p>
-    {% blocktrans trimmed %}
-      Without any apps, your {{ box_name }} cannot do very much.
-    {% endblocktrans %}
-  </p>
+  <ol class='next-steps'>
+    <li>
+      <div class="app-icon fa fa-refresh"></div>
+      <div class="step-text">
+        {% url 'upgrades:index' as upgrades_url %}
+        {% blocktrans trimmed %}
+          Automatic <a href="{{ upgrades_url }}" target="_blank">software
+          update</a> runs daily by default. For the first time, manually run
+          it now.
+        {% endblocktrans %}
+        <form class="form" method="post" target="_blank"
+              action="{% url 'upgrades:upgrade' %}">
+          {% csrf_token %}
+          <input type="submit" class="btn btn-default"
+                 value="{% trans "Update now" %}"/>
+        </form>
+      </div>
+    </li>
 
-  <div class="text-center">
-    <a class="btn btn-lg btn-primary" href="{% url 'apps' %}">
-      {% trans "Install Apps" %}</a>
-  </div>
+    <li>
+      <div class="app-icon fa fa-eye-slash"></div>
+      <div class="step-text">
+        {% url 'privacy:index' as privacy_url %}
+        {% blocktrans trimmed %}
+          Review <a href="{{ privacy_url }}" target="_blank">privacy options</a>.
+        {% endblocktrans %}
+      </div>
+    </li>
 
-  <p>
-    <p>
-      {% url 'networks:index' as networks_url %}
-      {% blocktrans trimmed %}
-        You may want to check the <a href="{{ networks_url }}">network
-        setup</a> and modify it if necessary.
-      {% endblocktrans %}
-    </p>
-  </p>
+    <li>
+      <div class="app-icon fa fa-signal"></div>
+      <div class="step-text">
+        {% url 'networks:index' as networks_url %}
+        {% blocktrans trimmed %}
+          Review and setup <a href="{{ networks_url }}" target="_blank">network
+          connections</a>. Change the default Wi-Fi password, if applicable.
+        {% endblocktrans %}
+      </div>
+    </li>
+
+    <li>
+      <div class="app-icon fa fa-tags"></div>
+      <div class="step-text">
+        {% url 'names:index' as names_url %}
+        {% blocktrans trimmed %}
+          Configure a <a href="{{ names_url }}" target="_blank">domain name</a>.
+        {% endblocktrans %}
+      </div>
+    </li>
+
+    <li>
+      <div class="app-icon fa fa-files-o"></div>
+      <div class="step-text">
+        {% url 'backups:index' as backups_url %}
+        {% blocktrans trimmed %}
+          Configure and schedule remote
+          <a href="{{ backups_url }}" target="_blank">backups</a>.
+        {% endblocktrans %}
+      </div>
+    </li>
+
+    <li>
+      <div class="app-icon fa fa-th"></div>
+      <div class="step-text">
+        {% url 'apps' as apps_url %}
+        {% blocktrans trimmed %}
+          Put {{ box_name }} to use by installing
+          <a href="{{ apps_url }}" target="_blank">apps</a>.
+        {% endblocktrans %}
+      </div>
+    </li>
+  </ul>
 
 {% endblock %}
diff --git a/static/themes/default/css/main.css b/static/themes/default/css/main.css
index 5492689f8..b8aecfe08 100644
--- a/static/themes/default/css/main.css
+++ b/static/themes/default/css/main.css
@@ -636,6 +636,29 @@ a.menu_link_active {
     width: 50%;
 }
 
+.next-steps {
+    list-style: none;
+    margin-top: 1.5rem;
+    padding: 0;
+}
+
+.next-steps li {
+    display: flex;
+    align-items: center;
+    padding: 0.75rem 0;
+}
+
+.next-steps .app-icon {
+    font-size: 3rem;
+    margin-right: 1rem;
+    width: 3rem;
+    text-align: center;
+}
+
+.next-steps form {
+    display: inline;
+}
+
 /*
  * Toggle button
  */