diff --git a/app/helpers/accounts_helper.rb b/app/helpers/accounts_helper.rb index a119246fd..d35ed07ed 100644 --- a/app/helpers/accounts_helper.rb +++ b/app/helpers/accounts_helper.rb @@ -36,7 +36,7 @@ module AccountsHelper end [ - family.build_cache_key("account_sidebar_tabs_v1", invalidate_on_data_updates: true), + family.build_cache_key("account_sidebar_tabs_v2", invalidate_on_data_updates: true), Current.user&.id, shares_version, active_tab, diff --git a/app/views/accounts/_account_sidebar_tabs.html.erb b/app/views/accounts/_account_sidebar_tabs.html.erb index 653a64c85..9b9087091 100644 --- a/app/views/accounts/_account_sidebar_tabs.html.erb +++ b/app/views/accounts/_account_sidebar_tabs.html.erb @@ -3,22 +3,12 @@ <% cache account_sidebar_tabs_cache_key(family: family, active_tab: active_tab, mobile: mobile), expires_in: 12.hours do %>
<% if family.missing_data_provider? %> -
- -
- <%= icon "triangle-alert", size: "sm", color: "warning" %> -

<%= t("accounts.sidebar.missing_data") %>

-
- - <%= icon("chevron-down", color: "warning", class: "group-open:transform group-open:rotate-180") %> -
-
-

<%= t("accounts.sidebar.missing_data_description", product: product_name) %>

-

- <%= link_to t("accounts.sidebar.configure_providers"), settings_hosting_path, class: "text-yellow-600 underline" %> -

-
-
+
+ <%= render DS::Alert.new(title: t("accounts.sidebar.missing_data"), variant: :warning) do %> +

<%= t("accounts.sidebar.missing_data_description", product: product_name) %>

+

<%= link_to t("accounts.sidebar.configure_providers"), settings_hosting_path, class: "underline font-medium" %>

+ <% end %> +
<% end %> <%= render DS::Tabs.new(active_tab: active_tab, session_key: "account_sidebar_tab", testid: "account-sidebar-tabs") do |tabs| %> diff --git a/app/views/import/confirms/_mappings.html.erb b/app/views/import/confirms/_mappings.html.erb index 2cfb6ab46..9ad032903 100644 --- a/app/views/import/confirms/_mappings.html.erb +++ b/app/views/import/confirms/_mappings.html.erb @@ -23,7 +23,7 @@ <% elsif import.has_unassigned_account? %>
-
+
<%= tag.p t(".unassigned_account"), class: "text-sm" %> <%= render DS::Link.new( text: t(".create_account"), diff --git a/app/views/rules/index.html.erb b/app/views/rules/index.html.erb index 4c43de864..57fc04e12 100644 --- a/app/views/rules/index.html.erb +++ b/app/views/rules/index.html.erb @@ -147,17 +147,11 @@
<% if run.pending? %> - - <%= t("rules.recent_runs.statuses.#{run.status}") %> - + <%= render DS::Pill.new(label: t("rules.recent_runs.statuses.#{run.status}"), tone: :warning, marker: false) %> <% elsif run.success? %> - - <%= t("rules.recent_runs.statuses.#{run.status}") %> - + <%= render DS::Pill.new(label: t("rules.recent_runs.statuses.#{run.status}"), tone: :success, marker: false) %> <% else %> - - <%= t("rules.recent_runs.statuses.#{run.status}") %> - + <%= render DS::Pill.new(label: t("rules.recent_runs.statuses.#{run.status}"), tone: :error, marker: false) %> <% end %> <% if run.failed? && run.error_message.present? %>
diff --git a/app/views/simplefin_items/_replacement_prompt.html.erb b/app/views/simplefin_items/_replacement_prompt.html.erb index 4337a8f69..095601f9e 100644 --- a/app/views/simplefin_items/_replacement_prompt.html.erb +++ b/app/views/simplefin_items/_replacement_prompt.html.erb @@ -20,7 +20,7 @@ <% dismissal_key = "#{old_sfa.id}:#{new_sfa.id}" %> <% next if dismissed_ids.include?(dismissal_key) %> -
+
<%= icon "credit-card", size: "sm", color: "warning", class: "shrink-0 mt-0.5" %>
@@ -33,7 +33,7 @@ params: { dormant_sfa_id: old_sfa.id, active_sfa_id: new_sfa.id }, method: :post, form: { class: "inline shrink-0" }, - class: "-mt-1 -mr-1 p-1 rounded hover:bg-yellow-100 theme-dark:hover:bg-yellow-900/40 text-secondary", + class: "-mt-1 -mr-1 p-1 rounded hover:bg-warning/20 text-secondary", aria: { label: t("simplefin_items.replacement_prompt.dismiss_aria") } do %> <%= icon "x", size: "sm" %> <% end %>