Skip to content

TT-7300 Mark Verses highlighting & Edit Reference (#314)#322

Open
gtryus wants to merge 2 commits into
developfrom
TT-7300-mark-vrs-hilite
Open

TT-7300 Mark Verses highlighting & Edit Reference (#314)#322
gtryus wants to merge 2 commits into
developfrom
TT-7300-mark-vrs-hilite

Conversation

@gtryus
Copy link
Copy Markdown
Contributor

@gtryus gtryus commented May 21, 2026

This PR improves Mark Verses on mobile and desktop so the waveform, table, and Edit Reference dialog stay aligned with the wireframes and behave correctly for partial marking, reference editing, and navigation.

  • Table - waveform sync & selection
  • Setment status colors (green / yellow / gray)
  • Edit Reference dialog

- Improved segment matching logic in usePlayerLogic for better accuracy.
- Updated WSAudioPlayer to support new markVersesTailOpenRef prop for managing verse states.
- Enhanced PassageDetailMarkVerses to handle incomplete verse rows and apply region colors dynamically.
- Refactored EditReferenceDropdown to manage end verse options and suffix handling more effectively.
- Improved MarkVersesTableIsMobile to allow for better interaction and visual feedback based on segment states.
- Added tests to ensure correct behavior of new features and UI changes.
@gtryus gtryus changed the title TT-7335 Improve aesthetics of racetrack UI (#314) Mark Verses highlighting & Edit Reference (#314) May 21, 2026
@gtryus gtryus changed the title Mark Verses highlighting & Edit Reference (#314) TT-7300 Mark Verses highlighting & Edit Reference (#314) May 21, 2026
@gtryus gtryus requested a review from sarahentzel May 21, 2026 21:39
- Updated existing test to verify that the reference input is not read-only when editable.
- Added new test to ensure the reference input remains read-only and disabled when the row lacks timestamps.
- Improved interaction feedback for better user experience in mobile view.
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.

1 participant