Skip to content

Conversation

@Alex-Tideman
Copy link
Collaborator

@Alex-Tideman Alex-Tideman commented Oct 23, 2025

Description & motivation 💭

Replace the green/red stripes with gradient

Screenshots (if applicable) 📸

Current
Screenshot 2025-10-23 at 4 47 08 PM

Update
Screenshot 2025-10-23 at 4 41 10 PM
Screenshot 2025-10-23 at 4 41 23 PM
Screenshot 2025-10-23 at 4 43 00 PM
Screenshot 2025-10-23 at 4 43 11 PM

Design Considerations 🎨

Testing 🧪

How was this tested 👻

  • Manual testing
  • E2E tests added
  • Unit tests added

Steps for others to test: 🚶🏽‍♂️🚶🏽‍♀️

Checklists

Draft Checklist

  • Validate where all the green/red Timeline updates will go

Merge Checklist

Issue(s) closed

Docs

Any docs updates needed?

@vercel
Copy link

vercel bot commented Oct 23, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
holocene Ready Ready Preview Comment Oct 24, 2025 1:53pm

@Alex-Tideman Alex-Tideman marked this pull request as draft October 23, 2025 21:59
.ContinuedAsNew {
@apply fill-green-200;
}
</style>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rip and tear 🤘🏼

<div
class="h-full w-full"
style="background: linear-gradient(255deg, #1FF1A5 0%, #F55 100%);"
></div>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you leave a brief synopsis of why this instead of a <linearGradient> stroke or sumpin

Copy link
Collaborator Author

@Alex-Tideman Alex-Tideman Oct 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

defs aren't being respected on the line, I think because of how deeply nested the timeline elements are and the fact that I used <g> groups and <foreignObjects>. Basically the definition of linearGradient isn't applied because it's overridden somewhere in the svg. Tried moving it to a bunch of different locations, but nothing worked. So... opted for simplicity with a little hit of a DOM node.

@Alex-Tideman Alex-Tideman merged commit c90bbe8 into main Oct 24, 2025
14 checks passed
@Alex-Tideman Alex-Tideman deleted the timeline-retry-gradient branch October 24, 2025 18:57
temporal-cicd bot pushed a commit that referenced this pull request Nov 4, 2025
Auto-generated version bump from 2.42.1 to 2.43.0

Bump type: minor

Changes included:
- [`c772936b`](c772936) add new logos (#2950)
- [`e4659afc`](e4659af) Add TLS support for ui-server (#2745)
- [`35333bc0`](35333bc) remove scroll up/down buttons in main content (#2975)
- [`25f4cd50`](25f4cd5) Bump sveltekit-superforms from 2.27.1 to 2.27.4 (#2965)
- [`133577d5`](133577d) Change sticky header to surface-background and include border (#2979)
- [`c90bbe85`](c90bbe8) Add gradient for completed Activities with retries (#2977)
- [`015de88a`](015de88) Set day as max unit in formatDate and getDuration (#2978)
- [`edf98d97`](edf98d9) Allow custom keyword SAs to be quick filterable (#2980)
- [`6183f331`](6183f33) Account for units less than a day in formatDate (#2982)
- [`2688adac`](2688ada) Hide timeout if 0s (#2983)
- [`7732e4b8`](7732e4b) feat(search-attributes): add disableTypeForExisting prop (#2984)
- [`66a6323f`](66a6323) Fix holocene component prop types (#2985)
- [`c8673e6c`](c8673e6) Fix handleFocusOut on maximize to allow highlighting text to copy (#2986)
- [`547045af`](547045a) SVG loading (#2955)
- [`ecacb1e0`](ecacb1e) fix(search-attributes): use $derived for reactive props pattern (#2992)
- [`9f00bdf3`](9f00bdf) Use CurrentVersion createTime (#2995)
- [`6bf4c09c`](6bf4c09) DT-2423 - move theme selector to user dropdown menu (#2988)
- [`49746aba`](49746ab) DT-3415 - move data encoder settings for into a modal (#2981)
- [`8046e1ac`](8046e1a) Add laptop-code and shield icons to Holocene icon system (#2997)
- [`05d8686b`](05d8686) Add a slot to the bottom nav for user profile picture in cloud (#3004)
- [`18d60d43`](18d60d4) Use page.data.namespace to access namespaceInfo (#3002)
- [`2f24a1b5`](2f24a1b) Task Failure System View (#2991)
laurakwhit added a commit that referenced this pull request Nov 4, 2025
Auto-generated version bump from 2.42.1 to 2.43.0

Bump type: minor

Changes included:
- [`c772936b`](c772936) add new logos (#2950)
- [`e4659afc`](e4659af) Add TLS support for ui-server (#2745)
- [`35333bc0`](35333bc) remove scroll up/down buttons in main content (#2975)
- [`25f4cd50`](25f4cd5) Bump sveltekit-superforms from 2.27.1 to 2.27.4 (#2965)
- [`133577d5`](133577d) Change sticky header to surface-background and include border (#2979)
- [`c90bbe85`](c90bbe8) Add gradient for completed Activities with retries (#2977)
- [`015de88a`](015de88) Set day as max unit in formatDate and getDuration (#2978)
- [`edf98d97`](edf98d9) Allow custom keyword SAs to be quick filterable (#2980)
- [`6183f331`](6183f33) Account for units less than a day in formatDate (#2982)
- [`2688adac`](2688ada) Hide timeout if 0s (#2983)
- [`7732e4b8`](7732e4b) feat(search-attributes): add disableTypeForExisting prop (#2984)
- [`66a6323f`](66a6323) Fix holocene component prop types (#2985)
- [`c8673e6c`](c8673e6) Fix handleFocusOut on maximize to allow highlighting text to copy (#2986)
- [`547045af`](547045a) SVG loading (#2955)
- [`ecacb1e0`](ecacb1e) fix(search-attributes): use $derived for reactive props pattern (#2992)
- [`9f00bdf3`](9f00bdf) Use CurrentVersion createTime (#2995)
- [`6bf4c09c`](6bf4c09) DT-2423 - move theme selector to user dropdown menu (#2988)
- [`49746aba`](49746ab) DT-3415 - move data encoder settings for into a modal (#2981)
- [`8046e1ac`](8046e1a) Add laptop-code and shield icons to Holocene icon system (#2997)
- [`05d8686b`](05d8686) Add a slot to the bottom nav for user profile picture in cloud (#3004)
- [`18d60d43`](18d60d4) Use page.data.namespace to access namespaceInfo (#3002)
- [`2f24a1b5`](2f24a1b) Task Failure System View (#2991)

Co-authored-by: laurakwhit <15069288+laurakwhit@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants