Skip to content

feat: Sticky workflow header#1058

Merged
Assem-Uber merged 9 commits intocadence-workflow:masterfrom
Assem-Uber:feature/15953/sticky-workflow-header
Nov 4, 2025
Merged

feat: Sticky workflow header#1058
Assem-Uber merged 9 commits intocadence-workflow:masterfrom
Assem-Uber:feature/15953/sticky-workflow-header

Conversation

@Assem-Uber
Copy link
Contributor

@Assem-Uber Assem-Uber commented Oct 28, 2025

Summary
Support prop for workflow header to allow sticky behavior. Sticky header is disabled by default

Screen recording

Screen.Recording.2025-10-28.at.08.44.17.mov

const sentinel = sentinelRef.current;
if (!sentinel) return;

const observer = new IntersectionObserver(
Copy link
Member

Choose a reason for hiding this comment

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

Don't we have a React hook for the intersection observer, with mocks? We can use that instead

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, i was trying to recall what it was. But now i recall i just saw it in history page.

Signed-off-by: Assem Hafez <assem.hafez@uber.com>
@Assem-Uber Assem-Uber merged commit 07c58ce into cadence-workflow:master Nov 4, 2025
5 checks passed
@Assem-Uber Assem-Uber deleted the feature/15953/sticky-workflow-header branch November 4, 2025 13:22
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