TSA-265: Editing mode on the verification screen - #312
Conversation
IgnacioRB
left a comment
There was a problem hiding this comment.
I tested this locally. I could edit a page, press Ctrl+Enter to store the corrected text and mark the page as corrected, discard changes with Esc, and type shortcuts inside the text box without triggering unwanted actions. Just a few observations I encountered:
- Automatic focus: When entering edit mode, I noticed one of the acceptance criteria is that the whole flow works without the mouse. The only time I needed it was to click inside the text box to start writing. It would be better if activating edit mode automatically focuses on the text box without needing to click it.
- Missing edited transcriptions on reload: When I edited a page and saved with
Ctrl+Enter, I reloaded the site and noticed my edits disappeared and remained with the original transcriptions. In the database, they are still saved in theedited_textcolumn, but they don't show up on the verification screen after a reload.
Minor suggestions:
3. Navigation block while editing: While in edit mode, I can still switch pages using shortcuts if I click outside the text box. In a keyboard flow, the risk is low, but it could happen if the user accidentally clicks outside the text box and goes to another page, causing changes to be lost. It would be better to ignore navigation shortcuts (like arrow keys) while edit mode is active, or at least add a warning stating that changes will be lost to prevent users from accidentally switching pages.
4. Reset edit mode on page change: If we open edit mode on a page and navigate to the next one while editing, the edit mode stays active for the following pages. It would be preferred to reset edit mode if it's still active when the page changes.
…https://github.com/BinaryStudioAcademy/bsa-2026-transcripta into feat/TSA-265-editing-mode-on-the-verification-screen
…https://github.com/BinaryStudioAcademy/bsa-2026-transcripta into feat/TSA-265-editing-mode-on-the-verification-screen
|
Tested locally. Editing flow works as expected. |
Closes: #265
Summary
Implemented the frontend editing flow for the verification screen.
Changes
FE:
Ctrl+Enter/Cmd+Enterto save the correction and move to the next page.Escto cancel editing without saving.action: "correct"with the editedtext,transcriptionId, anddurationMs.409 Conflict:BE:
editedText ?? textwhen loading document pages to display corrected transcriptions after page reload.