Skip to content

Conversation

@haoqunjiang
Copy link
Member

@haoqunjiang haoqunjiang commented Nov 27, 2025

It seems easy, so I just vibe-coded this feature. Looks like it's working. Will do a self-review before marking it as ready.

https://deploy-preview-658--sefirot-story.netlify.app/story/stories-components-stable-01-playground-story-vue?variantId=_default

@netlify
Copy link

netlify bot commented Nov 27, 2025

Deploy Preview for sefirot-story ready!

Name Link
🔨 Latest commit d306d4e
🔍 Latest deploy log https://app.netlify.com/projects/sefirot-story/deploys/692ea599cd6b310008409e1b
😎 Deploy Preview https://deploy-preview-658--sefirot-story.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Nov 27, 2025

Deploy Preview for sefirot-docs ready!

Name Link
🔨 Latest commit d306d4e
🔍 Latest deploy log https://app.netlify.com/projects/sefirot-docs/deploys/692ea599d5434a00085a23a3
😎 Deploy Preview https://deploy-preview-658--sefirot-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@haoqunjiang haoqunjiang marked this pull request as ready for review November 28, 2025 06:16
@brc-dd
Copy link
Member

brc-dd commented Dec 2, 2025

Hmm, seems like the position is being calculated from the position before scrolling.

Screen.Recording.2025-12-02.at.4.27.57.PM.mov

@haoqunjiang
Copy link
Member Author

haoqunjiang commented Dec 2, 2025

Oops. Actually, it's over-correcting. Actually, we should not care about the scrollLeft as the indicator is positioned relative to the table anyway, and for the result, we only need the delta.

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