fix(ui): fix few common ui bugs

[Subash-Mohan]

Description

• Fixed expandable text streaming display by replacing scroll-based approach with CSS transform snapping to element
  boundaries, preventing partial block clipping and scroll jitter
• Fixed button height inconsistencies in parallel/streaming timeline headers by adding proper size props
• Fixed SearchChipList unnecessary re-renders by replacing useState/useEffect state management with useMemo derivation and
  simple visibleCount state
• Fixed tab trigger text spacing and source tag description card padding

How Has This Been Tested?

Tested all he above scenarios

Additional Options

• [Required] I have considered whether this PR needs to be cherry-picked to the latest beta branch.
• [Optional] Override Linear Check

---

Summary by cubic

Fixes streaming text cutoff and jitter by snapping content to block boundaries with CSS transforms, and aligns button sizing and spacing for cleaner timelines. Also reduces SearchChipList re-renders to improve timeline performance.

• Bug Fixes
  • Streaming text: replaced scroll-to-bottom with transform-based snap to block boundaries to prevent clipping and flicker; accurate truncation detection for streaming/static content with a top ellipsis.
  • Timeline headers: standardized expand/collapse button size (sm) and transparent background to fix height mismatches.
  • SearchChipList: derive display via useMemo with a simple visibleCount; removed batch state and useEffects; stabilized a single “more” key to avoid re-renders.
  • UI spacing: added small padding for tab trigger text and SourceTagDetailsCard descriptions for consistent layout.

^{Written for commit c3e69db. Summary will update on new commits.}

[github-actions]

Preview Deployment

Status	Preview	Commit	Updated
✅	https://onyx-preview-pee8ybhaq-danswer.vercel.app	c3e69db	2026-02-16 11:06:05 UTC

[greptile-apps]

Greptile Overview

Greptile Summary

Fixed multiple UI consistency issues across timeline headers, expandable text display, and source tags:

• Replaced scroll-based truncation with CSS transform snapping in ExpandableTextDisplay to prevent partial block clipping during streaming
• Optimized SearchChipList by replacing useState/useEffect with useMemo derivation to eliminate unnecessary re-renders
• Added explicit size props to buttons in timeline headers for consistent heights
• Improved spacing and padding in tab triggers and source tag description cards

All changes are focused styling and performance improvements with no breaking changes or logic modifications.

Confidence Score: 4/5

• This PR is safe to merge with minor cleanup opportunity
• All changes are UI/styling improvements and performance optimizations. The ExpandableTextDisplay refactor is complex but well-structured. One minor cleanup suggestion for unused batchId field in SearchChipList.
• No files require special attention - all changes are straightforward UI improvements

Important Files Changed

Filename	Overview
web/src/app/app/message/messageComponents/timeline/headers/ParallelStreamingHeader.tsx	Added size="sm" prop and className="bg-transparent" to button for consistent styling
web/src/app/app/message/messageComponents/timeline/headers/StreamingHeader.tsx	Added size="md" prop to OpalButton for consistent height across header buttons
web/src/app/app/message/messageComponents/timeline/renderers/search/SearchChipList.tsx	Replaced useState/useEffect with useMemo for display list derivation, eliminating unnecessary re-renders. Minor cleanup opportunity for unused batchId field.
web/src/refresh-components/Tabs.tsx	Added px-0.5 wrapper div around text children for improved spacing consistency
web/src/refresh-components/buttons/source-tag/SourceTagDetailsCard.tsx	Wrapped description in div with px-1.5 pb-1 for consistent padding with rest of card
web/src/refresh-components/texts/ExpandableTextDisplay.tsx	Replaced scroll-based streaming display with CSS transform snapping to element boundaries, preventing partial block clipping. Complex but well-structured change that adds CONTAINER_TAGS set and snap algorithm logic.

_{Last reviewed commit: 824caab}

[greptile-apps]

_{6 files reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

[greptile-apps]

batchId field is no longer used and can be removed from the type

Suggested change

	chips.push({ type: "more", batchId: 0 });
	chips.push({ type: "more" });

[cubic-dev-ai]

No issues found across 6 files

[github-actions]

🖼️ Visual Regression Report

Project	Changed	Added	Removed	Unchanged	Report
admin	12	0	0	36	View Report
exclusive	0	0	0	0	No screenshots

[raunakab]

Approving! Just a few small comments here.

[raunakab]

I believe this one is supposed to be "md"?

[raunakab]

Approving this for now, but I will be throwing in some important LineItemLayout fixes which this component should use.