Skip to content

SF-3595 Fix scrolling of selected verse into view in the source pane - #4097

Merged
pmachapman merged 1 commit into
masterfrom
fix/SF-3595-source-verse-scroll
Sep 13, 2026
Merged

pmachapman merged 1 commit into
masterfrom
fix/SF-3595-source-verse-scroll

Conversation

@Nateowami

@Nateowami Nateowami commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

This change is Reviewable

@codecov

codecov Bot commented Sep 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.95%. Comparing base (fba2f08) to head (5b4d0cd).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4097      +/-   ##
==========================================
+ Coverage   80.94%   80.95%   +0.01%     
==========================================
  Files         664      664              
  Lines       42153    42153              
  Branches     6944     6917      -27     
==========================================
+ Hits        34121    34126       +5     
- Misses       6867     6879      +12     
+ Partials     1165     1148      -17     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@github-actions

github-actions Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

📸 Screenshot diff deployed! (1 change)

View the visual diff at: https://pr-4097--sf-screenshot-diffs.netlify.app

@Nateowami
Nateowami force-pushed the fix/SF-3595-source-verse-scroll branch from a9d3a9c to fac1083 Compare September 10, 2026 16:21
@Nateowami
Nateowami marked this pull request as ready for review September 11, 2026 02:47
@Nateowami Nateowami added testing complete Testing of PR is complete and should no longer hold up merging of the PR and removed ready to test labels Sep 11, 2026
@pmachapman
pmachapman self-requested a review September 13, 2026 21:27
@pmachapman pmachapman self-assigned this Sep 13, 2026
@pmachapman
pmachapman force-pushed the fix/SF-3595-source-verse-scroll branch from fac1083 to 5b4d0cd Compare September 13, 2026 21:27

@pmachapman pmachapman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

:lgtm:

@pmachapman reviewed 2 files and all commit messages, and made 1 comment.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on Nateowami).

@pmachapman
pmachapman deployed to screenshot_diff September 13, 2026 21:34 — with GitHub Actions Active
@pmachapman
pmachapman merged commit 568151e into master Sep 13, 2026
29 checks passed
@pmachapman
pmachapman deleted the fix/SF-3595-source-verse-scroll branch September 13, 2026 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing complete Testing of PR is complete and should no longer hold up merging of the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants