Skip to content

Scroll improvements - #75

Merged
Frequinzy merged 6 commits into
mainfrom
scroll-improvements
Apr 8, 2026
Merged

Scroll improvements#75
Frequinzy merged 6 commits into
mainfrom
scroll-improvements

Conversation

@Frequinzy

Copy link
Copy Markdown
Collaborator

This is a branch for improving scrolling.

The goal is to add so that you can sync position from preview to editor and vice-versa.

Add button to sync preview to cursor
Add so that preveiw scrolls when jumping to a scene from outline
Add automatic scrolling while live updating preview
@Frequinzy
Frequinzy requested a review from frblo April 5, 2026 11:16

@frblo frblo left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

There is a problem with when you go to a line in the fountain code and make a newline and start writing, where the preview was scrolled above the element you are currently editing. Then when you start editing the preview will automatically scroll to the line where you make the initial newline, meaning you can't see your new line.

Comment thread frontend/src/routes/(app)/+page.svelte Outdated
Comment thread frontend/src/routes/(app)/+page.svelte Outdated
Comment thread frontend/src/routes/(app)/+page.svelte Outdated
Comment thread frontend/src/lib/editor/Preview.svelte Outdated
Comment thread frontend/src/lib/editor/Editor.svelte Outdated

@frblo frblo left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I think it looks really good. However, I have a few notes which do not need to be fixed for this, but could be worth taking into consideration.

  • When clicking on a dialogue element you are taken to the character name, which may be intended?
  • Editing the title page does not scroll your preview, and it cannot be clicked och synced to/from.

@Frequinzy
Frequinzy marked this pull request as ready for review April 8, 2026 14:36

@frblo frblo left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGTM!

@Frequinzy
Frequinzy merged commit 22d69a5 into main Apr 8, 2026
1 check passed
@Frequinzy
Frequinzy deleted the scroll-improvements branch April 8, 2026 14:38
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.

2 participants