diff --git a/app/assets/images/system_user.png b/app/assets/images/system_user.png new file mode 100644 index 000000000..5204feb44 Binary files /dev/null and b/app/assets/images/system_user.png differ diff --git a/app/assets/stylesheets/card-columns.css b/app/assets/stylesheets/card-columns.css index 5fda5823b..2170b576e 100644 --- a/app/assets/stylesheets/card-columns.css +++ b/app/assets/stylesheets/card-columns.css @@ -438,9 +438,7 @@ /* Cover bubbles and cards on small viewports */ @media (max-width: 639px) { - &:not(.cards--closed &) { - margin-inline: calc(-1 * var(--cards-gap) - var(--main-padding)); - } + margin-inline: calc(-1 * var(--cards-gap) - var(--main-padding)); } &, .cards__filter { @@ -659,6 +657,7 @@ /* On Deck (Not Now) /* ------------------------------------------------------------------------ */ + .cards--closed, .cards--on-deck { --card-color: var(--color-ink-light) !important; @@ -671,6 +670,13 @@ } } + /* Closed (Done) + /* ------------------------------------------------------------------------ */ + + .cards--closed { + + } + /* Doing /* -------------------------------------------------------------------------- */ diff --git a/app/assets/stylesheets/print.css b/app/assets/stylesheets/print.css index 890e27ac3..c79493458 100644 --- a/app/assets/stylesheets/print.css +++ b/app/assets/stylesheets/print.css @@ -125,13 +125,11 @@ .card--new, .cards__decoration, .card-columns:before, - .cards--considering:before, - .cards--closed:before { + .cards--considering:before { display: none; } - .card-columns, - .cards--closed { + .card-columns { border-block-start: var(--border-light); margin-block-end: 1ch; min-block-size: unset; @@ -156,21 +154,6 @@ } } - .cards--closed { - .card:nth-of-type(7) { - margin-block: 0; - } - - .card:nth-of-type(n+7) { - border-inline: none; - max-inline-size: none; - } - - .card:nth-of-type(n+8) { - margin-block-start: calc(-1 * var(--cards-gap) - 1px); - } - } - /* Card perma /* ------------------------------------------------------------------------ */ diff --git a/app/controllers/users/avatars_controller.rb b/app/controllers/users/avatars_controller.rb index 9101ee401..61ce75f9f 100644 --- a/app/controllers/users/avatars_controller.rb +++ b/app/controllers/users/avatars_controller.rb @@ -7,7 +7,9 @@ class Users::AvatarsController < ApplicationController before_action :ensure_permission_to_administer_user, only: :destroy def show - if @user.avatar.attached? + if @user.system? + redirect_to view_context.image_path("system_user.png") + elsif @user.avatar.attached? redirect_to rails_blob_url(@user.avatar.variant(:thumb), disposition: "inline") elsif stale? @user, cache_control: cache_control render_initials diff --git a/app/helpers/events_helper.rb b/app/helpers/events_helper.rb index a29085d44..acc78da95 100644 --- a/app/helpers/events_helper.rb +++ b/app/helpers/events_helper.rb @@ -9,7 +9,7 @@ module EventsHelper "comment" when "card_title_changed" "rename" - when "card_board_changed", "card_triaged", "card_postponed" + when "card_board_changed", "card_triaged", "card_postponed", "card_auto_postponed" "move" else "person" diff --git a/app/helpers/webhooks_helper.rb b/app/helpers/webhooks_helper.rb index e0e0e3387..f3bb077c8 100644 --- a/app/helpers/webhooks_helper.rb +++ b/app/helpers/webhooks_helper.rb @@ -10,7 +10,7 @@ module WebhooksHelper card_closed: "Card moved to “Done”", card_reopened: "Card reopened", card_postponed: "Card moved to “Not Now”", - card_auto_postponed: "Card auto-closed as “Not Now”", + card_auto_postponed: "Card moved to “Not Now” due to inactivity", card_sent_back_to_triage: "Card moved back to “Maybe?”" }.with_indifferent_access.freeze diff --git a/app/models/card/eventable/system_commenter.rb b/app/models/card/eventable/system_commenter.rb index ace5701bf..0c9fe3517 100644 --- a/app/models/card/eventable/system_commenter.rb +++ b/app/models/card/eventable/system_commenter.rb @@ -25,7 +25,7 @@ class Card::Eventable::SystemCommenter when "card_postponed" "#{event.creator.name} moved this to “Not Now”" when "card_auto_postponed" - "Closed as “Not Now” due to inactivity" + "Moved to “Not Now” due to inactivity" when "card_title_changed" "#{event.creator.name} changed the title from “#{event.particulars.dig('particulars', 'old_title')}” to “#{event.particulars.dig('particulars', 'new_title')}”." when "card_board_changed" diff --git a/app/models/event/description.rb b/app/models/event/description.rb index 7718a795c..672784ddb 100644 --- a/app/models/event/description.rb +++ b/app/models/event/description.rb @@ -64,7 +64,7 @@ class Event::Description when "card_postponed" %(#{creator} moved #{card_title} to "Not Now") when "card_auto_postponed" - %(#{card_title} was closed as "Not Now" due to inactivity) + %(#{card_title} moved to "Not Now" due to inactivity) when "card_resumed" "#{creator} resumed #{card_title}" when "card_title_changed" diff --git a/app/models/user/day_timeline.rb b/app/models/user/day_timeline.rb index 623433126..b9ae4941a 100644 --- a/app/models/user/day_timeline.rb +++ b/app/models/user/day_timeline.rb @@ -38,7 +38,7 @@ class User::DayTimeline end def closed_column - @closed_column ||= build_column("Closed", 3, events.where(action: "card_closed")) + @closed_column ||= build_column("Done", 3, events.where(action: "card_closed")) end def cache_key @@ -55,6 +55,7 @@ class User::DayTimeline card_collection_changed card_board_changed card_postponed + card_auto_postponed card_triaged card_sent_back_to_triage comment_created diff --git a/app/models/user/role.rb b/app/models/user/role.rb index e12bf0191..0712e2cf3 100644 --- a/app/models/user/role.rb +++ b/app/models/user/role.rb @@ -10,7 +10,7 @@ module User::Role class_methods do def system - find_or_create_by!(role: :system) { it.name = "System" } + find_or_create_by!(account: Current.account, role: :system) { it.name = "System" } end end diff --git a/app/views/account/settings/_entropy.html.erb b/app/views/account/settings/_entropy.html.erb index 11ff9b326..ec9c58302 100644 --- a/app/views/account/settings/_entropy.html.erb +++ b/app/views/account/settings/_entropy.html.erb @@ -1,7 +1,7 @@

Auto close

-

Fizzy doesn’t let stale cards stick around forever. Cards automatically close as “Not Now” without activity for specific period of time. This is the default, global setting — you can override it on each board.

+

Fizzy doesn’t let stale cards stick around forever. Cards automatically move to “Not Now” if there is no activity for specific period of time. This is the default, global setting — you can override it on each board.

<%= render "entropy/auto_close", model: account.entropy, url: account_entropy_path, disabled: !Current.user.admin? %> diff --git a/app/views/boards/edit/_auto_close.html.erb b/app/views/boards/edit/_auto_close.html.erb index 2e4dc55a2..c32c1c050 100644 --- a/app/views/boards/edit/_auto_close.html.erb +++ b/app/views/boards/edit/_auto_close.html.erb @@ -1,7 +1,7 @@ <%= turbo_frame_tag @board, :entropy do %>

Auto close

-

Fizzy doesn’t let stale cards stick around forever. Cards automatically close as “Not now” if no one updates, comments, or moves a card for…

+

Fizzy doesn’t let stale cards stick around forever. Cards automatically move to “Not now” if no one updates, comments, or moves a card for…

<%= render "entropy/auto_close", model: board, url: board_entropy_path(board), disabled: !Current.user.can_administer_board?(@board) %>
<% end %> diff --git a/app/views/boards/show/_closed.html.erb b/app/views/boards/show/_closed.html.erb index 0d4c772f4..5d2820399 100644 --- a/app/views/boards/show/_closed.html.erb +++ b/app/views/boards/show/_closed.html.erb @@ -1,4 +1,4 @@ -<%= column_tag id: "closed-cards", name: "Done", drop_url: columns_card_drops_closure_path("__id__"), class: "cards--on-deck", style: "--card-color: var(--color-card-complete);", +<%= column_tag id: "closed-cards", name: "Done", drop_url: columns_card_drops_closure_path("__id__"), class: "cards--closed", style: "--card-color: var(--color-card-complete);", data: { drag_and_strum_target: "container", collapsible_columns_target: "column", diff --git a/app/views/boards/show/_mobile_columns.html.erb b/app/views/boards/show/_mobile_columns.html.erb index 0848b2cea..964d75220 100644 --- a/app/views/boards/show/_mobile_columns.html.erb +++ b/app/views/boards/show/_mobile_columns.html.erb @@ -32,7 +32,7 @@ <% end %> <% end %> - <%= link_to board_columns_closed_path(board), class: "cards cards--on-deck is-collapsed", style: "--card-color: var(--color-card-complete);", data: { turbo_frame: "_top" } do %> + <%= link_to board_columns_closed_path(board), class: "cards cards--closed is-collapsed", style: "--card-color: var(--color-card-complete);", data: { turbo_frame: "_top" } do %>
<%= board.cards.closed.count %>

diff --git a/app/views/cards/container/_closure.html.erb b/app/views/cards/container/_closure.html.erb index 6fa2fed57..f7c69a98d 100644 --- a/app/views/cards/container/_closure.html.erb +++ b/app/views/cards/container/_closure.html.erb @@ -12,7 +12,7 @@
<% if card.entropic? && card.open? && !card.postponed? %>
- Closes as “Not Now” <%= local_datetime_tag(card.entropy.auto_clean_at, style: :indays) -%> if there’s no activity. + Moves to “Not Now” <%= local_datetime_tag(card.entropy.auto_clean_at, style: :indays) -%> if there’s no activity.
<% end %>
diff --git a/app/views/public/boards/show/_closed.html.erb b/app/views/public/boards/show/_closed.html.erb index 9f1e1c930..ec75aed00 100644 --- a/app/views/public/boards/show/_closed.html.erb +++ b/app/views/public/boards/show/_closed.html.erb @@ -1,4 +1,4 @@ -