FIX: Summary update fixing a bug for TapInteraction

[josepmariapujol-unity]

Description

The interaction requires the control to be released within the timeout (the code uses <= durationOrDefault and cancels on timer expiry), so the comment should say "at most" / "within".

Testing status & QA

N/A

Overall Product Risks

Please rate the potential complexity and halo effect from low to high for the reviewers. Note down potential risks to specific Editor branches if any.

• Complexity: low
• Halo Effect: low

Comments to reviewers

Review carefully.

Checklist

Before review:

• Changelog entry added.
  • Explains the change in Changed, Fixed, Added sections.
  • For API change contains an example snippet and/or migration example.
  • JIRA ticket linked, example (case %%). If it is a private issue, just add the case ID without a link.
  • Jira port for the next release set as "Resolved".
• Tests added/changed, if applicable.
  • Functional tests Area_CanDoX, Area_CanDoX_EvenIfYIsTheCase, Area_WhenIDoX_AndYHappens_ThisIsTheResult.
  • Performance tests.
  • Integration tests.
• Docs for new/changed API's.
  • Xmldoc cross references are set correctly.
  • Added explanation how the API works.
  • Usage code examples added.
  • The manual is updated, if needed.

During merge:

• Commit message for squash-merge is prefixed with one of the list:
  • NEW: ___.
  • FIX: ___.
  • DOCS: ___.
  • CHANGE: ___.
  • RELEASE: 1.1.0-preview.3.

[u-pr-agent]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪ The PR contains a single-line documentation update in one file, making it extremely quick to review.

🏅 Score: 95 The change correctly addresses the semantic inaccuracy in the documentation regarding Tap interaction timing, though it introduces a minor grammatical redundancy.

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Redundant Phrasing The update results in the sentence: "Performs the action if the control is pressed and released within the set duration ... and then released." The final "and then released" (on line 14) is now redundant because "released" was added to line 12. Consider removing the trailing "and then released". /// Performs the action if the control is pressed and released within the set /// duration (which defaults to <see cref="InputSettings.defaultTapTime"/>) /// and then released.

Update review

---

🤖 Helpful? Please react with 👍/👎 | Questions❓Please reach out in Slack #ask-u-pr-agent

[u-pr-agent]

PR Code Suggestions ✨

No code suggestions found for the PR.

[codecov-github-com]

Codecov Report

All modified and coverable lines are covered by tests ✅

@@           Coverage Diff            @@
##           develop    #2308   +/-   ##
========================================
  Coverage    77.95%   77.95%           
========================================
  Files          476      476           
  Lines        97443    97443           
========================================
  Hits         75961    75961           
  Misses       21482    21482           

Flag	Coverage Δ
inputsystem_MacOS_2022.3	5.54% <ø> (ø)
inputsystem_MacOS_2022.3_project	75.47% <ø> (ø)
inputsystem_MacOS_6000.0	5.32% <ø> (ø)
inputsystem_MacOS_6000.0_project	77.37% <ø> (+<0.01%)	⬆️
inputsystem_MacOS_6000.2	5.32% <ø> (-0.01%)	⬇️
inputsystem_MacOS_6000.2_project	77.36% <ø> (ø)
inputsystem_MacOS_6000.3	5.32% <ø> (ø)
inputsystem_MacOS_6000.3_project	77.36% <ø> (-0.01%)	⬇️
inputsystem_MacOS_6000.4	5.32% <ø> (ø)
inputsystem_MacOS_6000.4_project	77.37% <ø> (+<0.01%)	⬆️
inputsystem_MacOS_6000.5	5.32% <ø> (ø)
inputsystem_MacOS_6000.5_project	77.37% <ø> (+<0.01%)	⬆️
inputsystem_Ubuntu_2022.3	5.54% <ø> (ø)
inputsystem_Ubuntu_2022.3_project	75.26% <ø> (ø)
inputsystem_Ubuntu_6000.0	5.32% <ø> (ø)
inputsystem_Ubuntu_6000.0_project	77.16% <ø> (ø)
inputsystem_Ubuntu_6000.2	5.32% <ø> (ø)
inputsystem_Ubuntu_6000.2_project	77.16% <ø> (+<0.01%)	⬆️
inputsystem_Ubuntu_6000.3	5.32% <ø> (ø)
inputsystem_Ubuntu_6000.3_project	77.16% <ø> (-0.01%)	⬇️
inputsystem_Ubuntu_6000.4	5.33% <ø> (ø)
inputsystem_Ubuntu_6000.4_project	77.18% <ø> (+<0.01%)	⬆️
inputsystem_Ubuntu_6000.5	5.33% <ø> (ø)
inputsystem_Ubuntu_6000.5_project	77.17% <ø> (ø)
inputsystem_Windows_2022.3	5.54% <ø> (ø)
inputsystem_Windows_2022.3_project	75.60% <ø> (ø)
inputsystem_Windows_6000.0	5.32% <ø> (ø)
inputsystem_Windows_6000.0_project	77.49% <ø> (ø)
inputsystem_Windows_6000.2	5.32% <ø> (ø)
inputsystem_Windows_6000.2_project	77.49% <ø> (ø)
inputsystem_Windows_6000.3	5.32% <ø> (ø)
inputsystem_Windows_6000.3_project	77.49% <ø> (+<0.01%)	⬆️
inputsystem_Windows_6000.4	5.32% <ø> (ø)
inputsystem_Windows_6000.4_project	77.49% <ø> (-0.01%)	⬇️
inputsystem_Windows_6000.5	5.32% <ø> (ø)
inputsystem_Windows_6000.5_project	77.49% <ø> (+<0.01%)	⬆️

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

Files with missing lines	Coverage Δ
...InputSystem/Actions/Interactions/TapInteraction.cs	67.18% <ø> (ø)

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.