Treat HistoryView item clicks as (de)select in selection mode

[jr]

Context

Address #2457 by changing click task container click action in selection mode to select/unselect rather than going to the task.

Screenshots

before

history-click-before.mov

after

history-click-after.mov

How to Test

• Go to chat history view
• Click Selection Mode button
• Click anywhere on a task
• Verify that it only selects the task

---

Important

In HistoryView, task item clicks in selection mode now toggle selection instead of opening the task, with updated tests and localization.

• Behavior:
  • In HistoryView.tsx, clicking a task item in selection mode now toggles selection instead of opening the task.
  • Adds data-testid to the selection mode toggle button for testing.
• Tests:
  • Adds test handles selection mode clicks in HistoryView.test.tsx to verify task selection toggling.
• Localization:
  • Updates deselectAll and selectAll text in history.json to lowercase.

^{This description was created by for 0c21c40. You can customize this summary. It will automatically update as commits are pushed.}

[changeset-bot]

⚠️ No Changeset found

Latest commit: 0c21c40

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[ellipsis-dev]

There's an inconsistency with the test ID naming: in most tests, the task item is referenced with virtuoso-item-1, but in the handles selection mode clicks test, it uses task-item-1. Please verify and update the test ID for consistency.

Suggested change

	const taskContainer = screen.getByTestId("task-item-1")
	const taskContainer = screen.getByTestId("virtuoso-item-1")

^{This comment was generated because it violated a code review rule: mrule_oAUXVfj5l9XxF01R.}

[jr]

virtuoso-item- is actually added in a mock, and doesn't have a click handler. I could change this to scope everything by virtuoso-item-1 and then click something within it, but I'm not sure that would have added much.