Skip to content

[DataEntryTable] Make recent entries read-only until clicked - #4373

Draft
imnasnainaec wants to merge 1 commit into
masterfrom
data-entry-narrow-recut
Draft

imnasnainaec wants to merge 1 commit into
masterfrom
data-entry-narrow-recut

Conversation

@imnasnainaec

@imnasnainaec imnasnainaec commented Sep 18, 2026

Copy link
Copy Markdown
Collaborator

Resolves #3493
Resolves #3813

🤖 Generated with Claude Code


This change is Reviewable

Recent-entry rows now render read-only (cold) by default; clicking a row
makes it editable (hot), and only one row is editable at a time. The row
returns to read-only when a gloss or vernacular edit is committed, when
the note is updated, or when the new check button is pressed.

This re-cuts the salvageable core of stale draft PR #3847 onto current
master; nothing else from that branch is carried over.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 18, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Sep 18, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 77.27273% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 67.13%. Comparing base (ba70f48) to head (d1c221c).

Files with missing lines Patch % Lines
src/components/DataEntry/DataEntryTable/index.tsx 60.00% 4 Missing ⚠️
...nents/DataEntry/DataEntryTable/RecentEntryCold.tsx 80.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4373      +/-   ##
==========================================
- Coverage   76.26%   67.13%   -9.13%     
==========================================
  Files         305      249      -56     
  Lines       11451     6338    -5113     
  Branches     1408      796     -612     
==========================================
- Hits         8733     4255    -4478     
+ Misses       2324     1821     -503     
+ Partials      394      262     -132     
Flag Coverage Δ
backend ?
frontend 67.13% <77.27%> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[DataEntry] Redesign for mobile view [DataEntry] Data entry difficult on mobile

1 participant