[ReviewEntries] Migrate from deprecations

[imnasnainaec]

Parts of #3180, #3879

Notes for review:

• In most places, replaced <Grid> with <Grid2>, <Stack>, <Box>, <div>, or nothing, as fit the design need.
• <Stack direction="row"> can be used in place of <Grid2> when we don't want the items in the row to wrap to a second row in a narrow window

Notes for testing:

• Do a side-by-side visual comparison between this and master (on QA) to make sure appearance is (roughly) the same or (subjectively) improved on all edited components
• Try each edited component with various window widths > 350px (we don't support narrower than that)
• Test with a few different UI languages

---

This change is 

[codecov]

Codecov Report

❌ Patch coverage is 56.25000% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.79%. Comparing base (72d059f) to head (7729988).
⚠️ Report is 6 commits behind head on master.

Files with missing lines	Patch %	Lines
...riesTable/Cells/EditCell/EditSensesCardContent.tsx	50.00%	3 Missing ⚠️
...s/ReviewEntriesTable/Cells/EditCell/EditDialog.tsx	60.00%	2 Missing ⚠️
...iewEntriesTable/Cells/EditCell/EditSenseDialog.tsx	0.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3883      +/-   ##
==========================================
- Coverage   73.85%   73.79%   -0.07%     
==========================================
  Files         289      289              
  Lines       10565    10561       -4     
  Branches     1326     1326              
==========================================
- Hits         7803     7793      -10     
- Misses       2376     2384       +8     
+ Partials      386      384       -2     

Flag	Coverage Δ
backend	85.41% <ø> (ø)
frontend	65.08% <56.25%> (-0.13%)	⬇️

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

☔ View full report in Codecov by Sentry.
📢 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.

[jasonleenaylor]

Reviewed 2 of 8 files at r2, 1 of 7 files at r3, 6 of 10 files at r4, 8 of 8 files at r5, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @imnasnainaec)

---

src/goals/ReviewEntries/ReviewEntriesTable/tests/index.test.tsx line 86 at r5 (raw file):

    await renderReviewEntriesTable();
    expect(mockGetFrontierWords).toHaveBeenCalled();
    const rowCount = mockWords().length + 1; // +1 for header row

Nice improvement.

---

src/goals/ReviewEntries/ReviewEntriesTable/Cells/EditCell/tests/EditSenseDialog.test.tsx line 139 at r5 (raw file):

      await renderEditSenseDialog(false, true);
      expect(screen.queryByText(definitionsTitle)).toBeNull();
      expect(screen.queryByText(partOfSpeechTitle)).toBeTruthy();

Technically this is less restrictive than before...

---

src/goals/ReviewEntries/ReviewEntriesTable/Cells/EditCell/tests/EditDialog.test.tsx line 23 at r5 (raw file):

  "goals/ReviewEntries/ReviewEntriesTable/Cells/EditCell/EditSensesCardContent",
  () => ({
    __esModule: true,

Fancy.

[imnasnainaec]

Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @imnasnainaec)

---

src/goals/ReviewEntries/ReviewEntriesTable/Cells/EditCell/tests/EditSenseDialog.test.tsx line 139 at r5 (raw file):

Previously, jasonleenaylor (Jason Naylor) wrote…

Technically this is less restrictive than before...

It will throw if the count is >1. queryAllBy... would be less restrictive.