mirror of
https://github.com/maybe-finance/maybe.git
synced 2025-08-09 23:45:21 +02:00
d75be2282b
Since the very first 0.1.0-alpha.1 release, we've been moving quickly to add new features to the Maybe app. In doing so, some parts of the codebase have become outdated, unnecessary, or overly-complex as a natural result of this feature prioritization. Now that "core" Maybe is complete, we're moving into a second phase of development where we'll be working hard to improve the accuracy of existing features and build additional features on top of "core". This PR is a quick overhaul of the existing codebase aimed to: - Establish the brand new and simplified dashboard view (pictured above) - Establish and move towards the conventions introduced in Cursor rules and project design overview #1788 - Consolidate layouts and improve the performance of layout queries - Organize the core models of the Maybe domain (i.e. Account::Entry, Account::Transaction, etc.) and break out specific traits of each model into dedicated concerns for better readability - Remove stale / dead code from codebase - Remove overly complex code paths in favor of simpler ones
98 lines
4 KiB
Text
98 lines
4 KiB
Text
<%# locals: (entry:, balance_trend: nil, view_ctx: "global") %>
|
|
|
|
<% transaction = entry.entryable %>
|
|
|
|
<%= turbo_frame_tag dom_id(entry) do %>
|
|
<%= turbo_frame_tag dom_id(transaction) do %>
|
|
<div class="grid grid-cols-12 items-center text-primary text-sm font-medium p-4
|
|
<%= @focused_record == entry || @focused_record == transaction ?
|
|
"border border-gray-900 rounded-lg" : "" %>">
|
|
|
|
<div class="pr-10 flex items-center gap-4
|
|
<%= balance_trend ? "col-span-6" : "col-span-8" %>">
|
|
<%= check_box_tag dom_id(entry, "selection"),
|
|
disabled: transaction.transfer?,
|
|
class: "checkbox checkbox--light",
|
|
data: {
|
|
id: entry.id,
|
|
"bulk-select-target": "row",
|
|
action: "bulk-select#toggleRowSelection"
|
|
} %>
|
|
|
|
<div class="max-w-full">
|
|
<%= content_tag :div, class: ["flex items-center gap-2"] do %>
|
|
<% if transaction.merchant&.icon_url %>
|
|
<%= image_tag transaction.merchant.icon_url,
|
|
class: "w-6 h-6 rounded-full",
|
|
loading: "lazy" %>
|
|
<% else %>
|
|
<%= render "shared/circle_logo",
|
|
name: entry.display_name,
|
|
size: "sm" %>
|
|
<% end %>
|
|
|
|
<div class="truncate">
|
|
<div class="space-y-0.5">
|
|
<div class="flex items-center gap-1">
|
|
<%= link_to(
|
|
transaction.transfer? ? transaction.transfer.name : entry.display_name,
|
|
transaction.transfer? ? transfer_path(transaction.transfer) : account_entry_path(entry),
|
|
data: {
|
|
turbo_frame: "drawer",
|
|
turbo_prefetch: false
|
|
},
|
|
class: "hover:underline hover:text-gray-800"
|
|
) %>
|
|
|
|
<% if entry.excluded %>
|
|
<span title="One-time <%= entry.amount.negative? ? "income" : "expense" %> (excluded from averages)">
|
|
<%= lucide_icon "asterisk", class: "w-4 h-4 shrink-0 text-orange-500" %>
|
|
</span>
|
|
<% end %>
|
|
|
|
<% if transaction.transfer? %>
|
|
<%= render "account/transactions/transfer_match", transaction: transaction %>
|
|
<% end %>
|
|
</div>
|
|
|
|
<div class="text-secondary text-xs font-normal">
|
|
<% if transaction.transfer? %>
|
|
<%= render "transfers/account_links",
|
|
transfer: transaction.transfer,
|
|
is_inflow: transaction.transfer_as_inflow.present? %>
|
|
<% else %>
|
|
<%= link_to entry.account.name,
|
|
account_path(entry.account, tab: "transactions", focused_record_id: entry.id),
|
|
data: { turbo_frame: "_top" },
|
|
class: "hover:underline" %>
|
|
<% end %>
|
|
</div>
|
|
</div>
|
|
</div>
|
|
<% end %>
|
|
</div>
|
|
</div>
|
|
|
|
<div class="flex items-center gap-1 col-span-2">
|
|
<%= render "account/transactions/transaction_category", transaction: transaction %>
|
|
</div>
|
|
|
|
<div class="col-span-2 ml-auto">
|
|
<%= content_tag :p,
|
|
transaction.transfer? && view_ctx == "global" ? "+/- #{format_money(entry.amount_money.abs)}" : format_money(-entry.amount_money),
|
|
class: ["text-green-600": entry.amount.negative?] %>
|
|
</div>
|
|
|
|
<% if balance_trend %>
|
|
<div class="col-span-2 justify-self-end">
|
|
<% if balance_trend.trend %>
|
|
<%= tag.p format_money(balance_trend.trend.current),
|
|
class: "font-medium text-sm text-primary" %>
|
|
<% else %>
|
|
<%= tag.p "--", class: "font-medium text-sm text-gray-400" %>
|
|
<% end %>
|
|
</div>
|
|
<% end %>
|
|
</div>
|
|
<% end %>
|
|
<% end %>
|