Skip to content

Conversation

reakaleek
Copy link
Member

@reakaleek reakaleek commented Feb 21, 2025

Closes #568

Context

When tabs are in a list or is a child of any block, then the index, which is used as ID is is wrong because it only searches for siblings.

Changes

This change uses the line number as ID for simplicity. It doesn't represent the actual number of the tab set but it's unique and effecient.

Result

cursorful-video-tabs-1740162192305.mp4

@reakaleek reakaleek self-assigned this Feb 21, 2025
@reakaleek reakaleek added the fix label Feb 21, 2025
@reakaleek reakaleek requested a review from a team February 21, 2025 18:20
@reakaleek reakaleek marked this pull request as ready for review February 21, 2025 18:24
Copy link
Member

@Mpdreamz Mpdreamz left a comment

Choose a reason for hiding this comment

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

Simple == brilliant

@reakaleek reakaleek enabled auto-merge (squash) February 22, 2025 09:56
@reakaleek reakaleek merged commit 11c2624 into main Feb 22, 2025
5 checks passed
@reakaleek reakaleek deleted the feature/fix-tabs branch February 22, 2025 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

tab groups can't be indented into lists
2 participants