Skip to content

fix(memory): timeline UI alignment issues and highlighting issue (#8753) to release v3.0#8797

Open
github-actions[bot] wants to merge 1 commit intorelease/v3.0from
hotfix/3bc1b89f-v3.0
Open

fix(memory): timeline UI alignment issues and highlighting issue (#8753) to release v3.0#8797
github-actions[bot] wants to merge 1 commit intorelease/v3.0from
hotfix/3bc1b89f-v3.0

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Feb 26, 2026

Cherry-pick of commit 3bc1b89 to release/v3.0 branch.

Original PR: #8753

  • [Optional] Override Linear Check

Summary by cubic

Fixes timeline alignment for reasoning/deep-research/memory tool steps and makes memory highlighting focus reliable in the Memories modal. Improves readability and interaction in v3.0.

  • Bug Fixes
    • Timeline: detect reasoning, deep research plan, and memory tool packets; render the active step full-width (no right padding) and pass noPaddingRight to the renderer.
    • Memories modal: after auto-scroll to a highlighted item, focus the textarea to ensure the highlight applies and allow immediate editing.

Written for commit d58ea04. Summary will update on new commits.

@github-actions github-actions bot requested a review from a team as a code owner February 26, 2026 08:54
@github-actions github-actions bot added the cherry-pick 🍒 Tags the PR to ensure that these are cherry-pick PRs label Feb 26, 2026
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 26, 2026

Greptile Summary

Cherry-pick of UI bug fixes from PR #8753 to the release/v3.0 branch.

Changes:

  • Fixed timeline alignment issues by removing right padding for reasoning, deep research, and memory tool content types
  • Improved memory modal UX by automatically focusing the textarea when highlighting a memory item

Code Quality:

  • All changes follow web/STANDARDS.md frontend standards
  • Proper use of React hooks with correct dependency arrays
  • TypeScript types are properly maintained
  • Uses existing helper functions from packetHelpers.ts

Confidence Score: 5/5

  • This PR is safe to merge with no risks
  • Clean cherry-pick of well-tested UI fixes with no logical errors, proper React patterns, and adherence to all frontend standards
  • No files require special attention

Important Files Changed

Filename Overview
web/src/app/app/message/messageComponents/timeline/ParallelTimelineTabs.tsx Added conditional padding logic for specific packet types to fix timeline alignment issues
web/src/refresh-components/modals/MemoriesModal.tsx Added textarea focus on highlight to improve UX when scrolling to memory items

Last reviewed commit: d58ea04

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-pick 🍒 Tags the PR to ensure that these are cherry-pick PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant