Skip to content

SF-3846 Prevent edits that modify text outside of the selected range#3986

Open
pmachapman wants to merge 1 commit into
masterfrom
fix/SF-3846
Open

SF-3846 Prevent edits that modify text outside of the selected range#3986
pmachapman wants to merge 1 commit into
masterfrom
fix/SF-3846

Conversation

@pmachapman

@pmachapman pmachapman commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

This PR prevents overwriting of verse markers by restricting keyboard input when text is selected outside of the current segment.


This change is Reviewable

@pmachapman pmachapman added the will require testing PR should not be merged until testers confirm testing is complete label Jul 8, 2026
@pmachapman pmachapman temporarily deployed to screenshot_diff July 8, 2026 01:22 — with GitHub Actions Inactive
@codecov

codecov Bot commented Jul 8, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 50.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.02%. Comparing base (4e33291) to head (44e3231).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...re/ClientApp/src/app/shared/text/text.component.ts 50.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3986      +/-   ##
==========================================
- Coverage   81.02%   81.02%   -0.01%     
==========================================
  Files         644      644              
  Lines       41445    41449       +4     
  Branches     6726     6752      +26     
==========================================
+ Hits        33580    33582       +2     
- Misses       6770     6772       +2     
  Partials     1095     1095              

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

will require testing PR should not be merged until testers confirm testing is complete

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant