Commit Graph

15 Commits

Author SHA1 Message Date
Mike Dalessio 9c76002bc1 Completing a step removes stalled status from UI
Add `updatedAt` to stalled bubble options so JS can check if the card
was recently updated, matching the `Card::Stallable#stalled?`
logic. Refresh the bubble via Turbo Stream when a step is updated.

This should have been part of #1625

Fixes https://app.fizzy.do/5986089/cards/3668
2026-01-16 09:58:21 -05:00
Jorge Manrubia effb96537a Use buil-in support for :self in Stimulus
Thanks to @brunoprietog for advice https://github.com/basecamp/fizzy/pull/1927/files/9639e07ab88f628f5f37ad2aa9acee2b86d4fb1a#diff-24a46f2ca97f2964d4d8940123b7f76578694567df84256f576aa4bee221179e
2025-12-04 22:16:00 +01:00
Jorge Manrubia 9639e07ab8 Save a bunch of invocations on morph events from children elements 2025-12-04 21:27:22 +01:00
Jorge Manrubia 704b5b9788 Fix: memoization was showing stale values when morphing 2025-12-04 21:21:54 +01:00
Jorge Manrubia 15d3c0e724 Simplify since the design is identical 2025-06-06 09:19:51 +02:00
Jorge Manrubia f742749b0f Tidy up controller and make sure the different targets are properly separated
If not, it wouldn't show stalled if there were entropy bubbles in the page
2025-06-06 09:15:51 +02:00
Jorge Manrubia 6287acf5ba Tidy up controller
Remove namespace "entropy" since the regions are common now
2025-06-06 09:04:42 +02:00
Jason Zimdars 9d706902e8 Style stalled bubble 2025-06-05 16:25:21 -05:00
Jorge Manrubia 6e8087ee2e Toggle entropy/stalled container in the bubbles controller 2025-06-05 11:08:20 +02:00
Jorge Manrubia bfa4dc626b Restructure to make it easy to extend rendering logic 2025-06-05 10:39:51 +02:00
Jorge Manrubia 76119b17bd Entropy, not bubble! 2025-06-05 10:33:02 +02:00
Jorge Manrubia 22ed605884 Namespace bubble-related targets 2025-06-05 10:32:14 +02:00
Jorge Manrubia 2cbd451944 Group entropy options in an object value
We'll add stalling considerations next, this helps keeping things organized.
2025-06-05 10:28:33 +02:00
Mike Dalessio dddb33b483 Use a view helper to remove branching from the bubbles controller 2025-05-16 10:32:28 -04:00
Mike Dalessio 388fe2de73 Introduce a stimulus controller for bubbles
The "closing soon" bubbles were introduced in #406, and the "falling
back" bubbles in #500. However, these bubbles are part of the cached
card and so as time passes, the relative time doesn't change unless
the card is touched.

This PR introduces a stimulus controller for bubbles, which takes care
of:

- making the bubble visible during the reminder period
- calculating the relative number of days until entropy kicks in
- rendering appropriate text around the day count

ref: https://37s.fizzy.37signals.com/collections/693169850/cards/999009091
2025-05-16 10:27:00 -04:00