diff --git a/app/assets/stylesheets/cards.css b/app/assets/stylesheets/cards.css index 1f7243315..b0e7ee481 100644 --- a/app/assets/stylesheets/cards.css +++ b/app/assets/stylesheets/cards.css @@ -63,7 +63,7 @@ .card__actions-container { background-color: var(--color-bg); - margin-block: -1em 1em; + margin-block-start: -1em; padding: 1em 2em; position: relative; z-index: 0; diff --git a/app/assets/stylesheets/comments.css b/app/assets/stylesheets/comments.css index ba89c6db6..58d6cd5df 100644 --- a/app/assets/stylesheets/comments.css +++ b/app/assets/stylesheets/comments.css @@ -1,20 +1,31 @@ .comments { --avatar-size: 2.33em; - --comment-padding-block: 0.25lh; - --comment-padding-inline: 3ch; + --comment-padding-block: 0.33lh; + --comment-padding-inline: 2ch; + display: flex; font-size: 1.33cqi; - inline-size: min(67ch, 100dvw); - max-inline-size: 100dvw; - row-gap: var(--comment-padding-block); + gap: var(--comment-padding-block); + inline-size: 100%; + max-inline-size: 60cqi; @media (max-width: 70ch) { - inline-size: 100%; + max-inline-size: 100%; } +} - > :first-child { - /* margin-block-start: var(--comment-padding-block); */ - } +.comments__column { + display: flex; + flex-direction: column; + flex: 4 1 80%; + gap: var(--comment-padding-block); +} + +.events__column { + display: flex; + flex-direction: column; + flex: 1 0 20%; + gap: 1lh; } .comment { @@ -23,8 +34,16 @@ position: relative; } -.comment--new { - border-block: 1px solid var(--color-subtle-dark); +.comment--new, +.comment--edit { + house-md-toolbar { + border-block-end: 1px solid var(--color-subtle-dark); + } + + house-md { + border-block: 1px solid var(--color-subtle-dark); + border-radius: 0; + } } .comment__avatar { @@ -59,15 +78,12 @@ .comment__content { margin-inline: auto 0 auto var(--avatar-size); + max-inline-size: 100%; padding: var(--comment-padding-block) var(--comment-padding-inline); .comment--mine_ & { margin-inline: var(--avatar-size) auto; } - - .comment:has(+ .event-summary) & { - padding-block-end: calc(var(--comment-padding-block) * 1.5); - } } .comment__input { @@ -98,20 +114,15 @@ margin-block-end: 0.3lh; } -.comment__submit { - inset: auto 0.5em 0.5em auto; - position: absolute; - z-index: 0; -} - .comment__timestamp { color: var(--color-subtle-dark); } .event-summary { color: var(--bubble-color); - font-weight: normal; + font-size: 0.8em; + font-weight: 500; gap: 0.5ch; opacity: 0.7; - padding: calc(var(--comment-padding-block) * 0.3) var(--comment-padding-inline); + text-align: start; } diff --git a/app/assets/stylesheets/house.css b/app/assets/stylesheets/house.css index aa959a4bf..c1ebba2b1 100644 --- a/app/assets/stylesheets/house.css +++ b/app/assets/stylesheets/house.css @@ -13,7 +13,6 @@ house-md { /* Toolbar */ house-md-toolbar { background-color: inherit; - border-block-end: 1px solid var(--color-subtle-dark); border-radius: 0; color: currentColor; display: inline-flex; diff --git a/app/helpers/messages_helper.rb b/app/helpers/messages_helper.rb index bac9af789..d66756318 100644 --- a/app/helpers/messages_helper.rb +++ b/app/helpers/messages_helper.rb @@ -1,7 +1,7 @@ module MessagesHelper def messages_tag(bubble, &) turbo_frame_tag dom_id(bubble, :messages), - class: "comments align-center center borderless margin flex flex-column gap-half", + class: "comments center borderless gap-half", style: "--bubble-color: <%= bubble.color %>", role: "group", aria: { label: "Messages" }, data: { diff --git a/app/models/event_summary.rb b/app/models/event_summary.rb index 6891da43d..221e78d2e 100644 --- a/app/models/event_summary.rb +++ b/app/models/event_summary.rb @@ -18,11 +18,11 @@ class EventSummary < ApplicationRecord def summarize(event) case event.action when "published" - "Added by #{event.creator.name} #{time_ago_in_words(event.created_at)} ago." + "Added by #{event.creator.name}." when "assigned" - "Assigned to #{event.assignees.pluck(:name).to_sentence} #{time_ago_in_words(event.created_at)} ago." + "Assigned to #{event.assignees.pluck(:name).to_sentence}." when "unassigned" - "Unassigned from #{event.assignees.pluck(:name).to_sentence} #{time_ago_in_words(event.created_at)} ago." + "Unassigned from #{event.assignees.pluck(:name).to_sentence}." when "staged" "#{event.creator.name} moved this to '#{event.stage_name}'." when "popped" diff --git a/app/views/bubbles/_messages.html.erb b/app/views/bubbles/_messages.html.erb index 1e72e908d..f8b656034 100644 --- a/app/views/bubbles/_messages.html.erb +++ b/app/views/bubbles/_messages.html.erb @@ -1,14 +1,22 @@ <%= messages_tag(bubble) do %> <%# Template Dependency: comments/comment %> <%# Template Dependency: event_summaries/event_summary %> - <%= render bubble.messages, cached: true %> - <%= render "comments/new", bubble: bubble %> + <%#= render bubble.messages, cached: true %> -
<%= pluralize(bubble.watchers.count, "person") %> will be notified when someone comments on this.
-
<%= pluralize(@bubble.watchers.count, "person") %> will be notified when someone comments on this.
+