Refactored the settings page into multiple pages

The previous page was becoming too long, now we have a sidebar with
separate sections so it is easier to understand.

Author: dutow

app/assets/stylesheets/components/settings.css

@@ -131,3 +131,52 @@
 .settings-google-svg {
   display: block;
 }
+
+/* Settings sidebar navigation */
+.settings-nav {
+  display: flex;
+  flex-direction: column;
+  gap: var(--spacing-2);
+  padding: var(--spacing-4) 0;
+}
+
+.settings-nav-link {
+  display: flex;
+  align-items: center;
+  gap: var(--spacing-3);
+  padding: var(--spacing-3) var(--spacing-4);
+  border-radius: var(--border-radius-md);
+  text-decoration: none;
+  color: var(--color-text-primary);
+  transition: background-color var(--transition-fast);
+  font-weight: var(--font-weight-medium);
+}
+
+.settings-nav-link:hover {
+  background-color: var(--color-bg-hover);
+}
+
+.settings-nav-link.active {
+  background-color: var(--color-primary-100);
+  color: var(--color-primary-700);
+  font-weight: var(--font-weight-semibold);
+}
+
+.settings-nav-link.danger {
+  color: var(--color-danger-text);
+}
+
+.settings-nav-link.danger:hover {
+  background-color: var(--color-danger-bg-hover);
+}
+
+.settings-nav-link i {
+  width: 20px;
+  text-align: center;
+}
+
+.settings-nav-divider {
+  height: 1px;
+  background-color: var(--color-border);
+  margin: var(--spacing-2) 0;
+}

app/controllers/account_deletions_controller.rb

@@ -19,12 +19,12 @@ def google_oauth2
     if linking && current_user
       if identity
         if identity.user_id != current_user.id
-          return redirect_to settings_path, alert: 'That Google account is already linked to another user.'
+          return redirect_to settings_account_path, alert: 'That Google account is already linked to another user.'
         end
-        return redirect_to settings_path, notice: 'That Google account is already linked to your account.'
+        return redirect_to settings_account_path, notice: 'That Google account is already linked to your account.'
       else
         if Alias.by_email(email).where.not(user_id: [nil, current_user.id]).exists?
-          return redirect_to settings_path, alert: 'Email is linked to another account. Delete that account first to release it.'
+          return redirect_to settings_account_path, alert: 'Email is linked to another account. Delete that account first to release it.'
         end
 
         aliases = Alias.by_email(email).where(user_id: [nil, current_user.id])
@@ -53,7 +53,7 @@ def google_oauth2
       end
 
       identity.update!(last_used_at: Time.current, email: email, raw_info: auth.to_json)
-      return redirect_to settings_path, notice: 'Google account linked.'
+      return redirect_to settings_account_path, notice: 'Google account linked.'
     end
 
     if identity

app/controllers/emails_controller.rb

@@ -1,6 +1,4 @@
 class PasswordsController < ApplicationController
-  before_action :require_authentication, only: [:update_current]
-
   def new
   end
 
@@ -32,19 +30,4 @@ def update
       render :edit, status: :unprocessable_entity
     end
   end
-
-  def update_current
-    user = current_user
-    if user.password_digest.present?
-      unless user.authenticate(params[:current_password])
-        return redirect_to settings_path, alert: "Current password is incorrect"
-      end
-    end
-
-    if user.update(password: params[:password], password_confirmation: params[:password_confirmation])
-      redirect_to settings_path, notice: "Password updated."
-    else
-      redirect_to settings_path, alert: user.errors.full_messages.to_sentence
-    end
-  end
 end

app/controllers/omniauth_callbacks_controller.rb

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+module Settings
+  class AccountsController < Settings::BaseController
+    def show
+      @aliases = current_user.person&.aliases&.order(:email) || []
+      @identities = current_user.identities.order(:provider, :email, :uid)
+      @default_alias_id = current_user.person&.default_alias_id
+    end
+
+    private
+
+    def active_settings_section
+      :account
+    end
+  end
+end

app/controllers/passwords_controller.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+module Settings
+  class BaseController < ApplicationController
+    before_action :require_authentication
+    layout 'settings'
+
+    helper_method :active_settings_section
+
+    private
+
+    def active_settings_section
+      :account
+    end
+  end
+end

app/controllers/read_status_imports_controller.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+module Settings
+  class DeletionsController < Settings::BaseController
+    def show
+    end
+
+    def create
+      unless params[:confirmation].to_s.strip == "DELETE"
+        return redirect_to settings_deletion_path, alert: "Please type DELETE to confirm."
+      end
+
+      perform_deletion!(current_user)
+      reset_session
+      redirect_to root_path, notice: 'Your account has been deleted.'
+    end
+
+    private
+
+    def active_settings_section
+      :deletion
+    end
+
+    def perform_deletion!(user)
+      Identity.where(user_id: user.id).delete_all
+      UserToken.where(user_id: user.id).delete_all
+      user.aliases.update_all(user_id: nil, verified_at: nil)
+      user.update!(password_digest: nil, deleted_at: Time.current)
+    end
+  end
+end