Skip to content

Conversation

@victorsanni
Copy link
Contributor

@victorsanni victorsanni commented May 22, 2024

Adds the click system sound and heavy-impact haptic feedback to iOS on long presses.

Fixes #148391

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@github-actions github-actions bot added the framework flutter/packages/flutter repository. See also f: labels. label May 22, 2024
@victorsanni victorsanni marked this pull request as ready for review May 22, 2024 23:48
@victorsanni victorsanni requested a review from justinmc May 24, 2024 21:21
Copy link
Contributor

@justinmc justinmc left a comment

Choose a reason for hiding this comment

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

LGTM with nits 👍

/// no-op as that platform usually doesn't provide feedback for long presses.
/// On Android the platform-typical vibration is triggered. On iOS a
/// heavy-impact haptic feedback is triggered, alongside the click system
/// sound.
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: "sound, which was observed to be the default behavior on a physical <iPhone version> running <iOS version>."

/// press.
/// On Android the platform-typical vibration is triggered. On iOS a
/// heavy-impact haptic feedback is triggered, alongside the click system
/// sound.
Copy link
Contributor

Choose a reason for hiding this comment

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

Probably also include the iOS device and version here.

Copy link

@JhonySpark JhonySpark left a comment

Choose a reason for hiding this comment

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

Good solution

@victorsanni victorsanni added the autosubmit Merge PR when tree becomes green via auto submit App label May 29, 2024
@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label May 30, 2024
@auto-submit
Copy link
Contributor

auto-submit bot commented May 30, 2024

auto label is removed for flutter/flutter/148922, due to - The status or check suite Linux web_long_running_tests_2_5 has failed. Please fix the issues identified (or deflake) before re-applying this label.

@victorsanni victorsanni added the autosubmit Merge PR when tree becomes green via auto submit App label May 30, 2024
@auto-submit auto-submit bot merged commit 3496b1d into flutter:master May 30, 2024
victorsanni added a commit to victorsanni/flutter that referenced this pull request May 31, 2024
Adds the click system sound and heavy-impact haptic feedback to iOS on long presses.

Fixes flutter#148391
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 2, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 2, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 3, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 3, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 3, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 3, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 3, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 3, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 3, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 3, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 3, 2024
@victorsanni victorsanni deleted the add-ios-feedback branch July 12, 2024 00:27
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feedback.forLongPress does not provide feedback on iOS

3 participants