Skip to content

Refactor drafting UI - #108

Merged
kaseywright merged 13 commits into
mainfrom
refactor-drafting-ui
Oct 16, 2025
Merged

Refactor drafting UI#108
kaseywright merged 13 commits into
mainfrom
refactor-drafting-ui

Conversation

@kaseywright

Copy link
Copy Markdown
Contributor

This refactor is to ensure alignment of the source verses and target verses when a translator is working on the drafting page.

The SourcePanel and TargetPanel components were kept for now. We can remove them after we are certain that we will not need them.

kaseywright added 2 commits October 8, 2025 09:26
This was necessary to ensure the alignment of the source and translated verses when the page was scrolled.
@kaseywright
kaseywright requested a review from vipinpaul October 8, 2025 14:15
Comment thread src/layouts/bible/DraftingPage.tsx Outdated
Comment thread src/layouts/bible/DraftingPage.tsx Outdated
Comment thread src/layouts/bible/DraftingPage.tsx
Comment thread src/layouts/bible/DraftingPage.tsx

{activeVerseId < totalSourceVerses && (
<div className='absolute right-4 z-10' style={{ top: buttonTop }}>
<Button

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.

Screenshot from 2025-10-13 17-49-46

The Next Verse button should be displayed at the bottom of the last verse.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

In the following, should the 'Next verse' button move to the following verse (1), the next empty verse (2), or the next verse after the last completed (3)?
If the button is anchored on the last completed verse, clicking it would be unintuitive.
Screenshot 2025-10-14 at 9 09 55 AM

@chadw-eten chadw-eten Oct 14, 2025

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

After discussing this, we have decided to keep the Next Verse button below the last verse that can be edited. The Next Verse button will be used only for "unlocking" the next verse for editing. The Enter or Tab keys will move the user down a verse. So, for example, if they translate all the way to verse 6 and then see a needed edit in verse 3, they can click on verse 3, make the edit, then press Enter three times to be back in the edit box for verse 6 and continue translating. If they hit Enter again when they are in the edit box for verse 6 or if they click the Next Verse button, it will move them to verse 7.

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.

Screenshot from 2025-10-13 17-49-46

The content should be stretched to use more horizontal space. There's space on both sides.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

There is a well understood design principle that text that is too wide is more difficult to read. The best width for text should be between 50-75 characters, or for blogs or larger bodies of text, the text area should be no more than roughly 700px. ref

The width I set is generally closer to 55 characters, depending on language. I can see an argument for a little wider text area here ( maybe up to max-w-7xl) but not filling the screen on larger devices.

@chadw-eten chadw-eten Oct 15, 2025

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I have always thought, like Kasey, that shorter line lengths are better. But I have done some research and I am finding that the shorter lengths apply to reading and not editing. For editing, having a larger context is better. Taking from other programs like Google Docs or MS Word they seem to be about 95 characters wide. I think Kasey's recommendation of 700px (or the widest preset width in Tailwind) for each the source and target pane will allow for that.

Comment thread src/layouts/bible/DraftingPage.tsx Outdated
@kaseywright
kaseywright requested a review from vipinpaul October 14, 2025 17:02
@kaseywright
kaseywright merged commit cf6b27b into main Oct 16, 2025
1 check passed
@kaseywright
kaseywright deleted the refactor-drafting-ui branch October 22, 2025 16:30
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.

3 participants