Skip to content

editor: Deduplicate sticky header rows#52844

Merged
cole-miller merged 2 commits intozed-industries:mainfrom
timvermeulen:deduplicate-sticky-header-rows
Mar 31, 2026
Merged

editor: Deduplicate sticky header rows#52844
cole-miller merged 2 commits intozed-industries:mainfrom
timvermeulen:deduplicate-sticky-header-rows

Conversation

@timvermeulen
Copy link
Copy Markdown
Contributor

Fixes a bug that caused duplicate sticky header rows to appear if multiple outline items start on the same row.

Sort of addresses #52722, although arguably the real issue there is that duplicate outline items are being created in the first place.

Before:

before.mov

After:

after.mov

Self-Review Checklist:

  • I've reviewed my own diff for quality, security, and reliability
  • Unsafe blocks (if any) have justifying comments
  • The content is consistent with the UI/UX checklist
  • Tests cover the new/changed behavior
  • Performance impact has been considered and is acceptable

Release Notes:

  • Fixed a bug that caused duplicate sticky header rows to appear if multiple outline items start on the same row.

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Mar 31, 2026
@zed-codeowner-coordinator zed-codeowner-coordinator bot requested review from a team and dinocosta and removed request for a team March 31, 2026 17:28
@github-actions github-actions bot added the community champion Issues filed by our amazing community champions! 🫶 label Mar 31, 2026
@cole-miller
Copy link
Copy Markdown
Member

Thanks!

@cole-miller cole-miller enabled auto-merge (squash) March 31, 2026 18:44
@cole-miller cole-miller merged commit 3a6faf2 into zed-industries:main Mar 31, 2026
33 checks passed
@timvermeulen timvermeulen deleted the deduplicate-sticky-header-rows branch April 1, 2026 23:57
@zed-zippy zed-zippy bot added the PR state:needs review Used to label PRs that are in need of a post-merge approval label Apr 6, 2026
@cole-miller
Copy link
Copy Markdown
Member

@zed-industries/approved

@MrSubidubi MrSubidubi removed the PR state:needs review Used to label PRs that are in need of a post-merge approval label Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement community champion Issues filed by our amazing community champions! 🫶

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants