amperfy icon indicating copy to clipboard operation
amperfy copied to clipboard

[Bug] Disabling haptic feedback has no effect

Open BLeeEZ opened this issue 1 year ago • 3 comments

@brokoler: please create for bugs and feature request separate tickets. It makes it easier for me to keep track of progression and makes discussions easier.

Minor issue: Disabling haptic feedback in latest beta has no effect, haptic feedback is active all the time.

BLeeEZ avatar Jun 21 '24 06:06 BLeeEZ

This option deactivates vibration for "Play" and "Shuffle" buttons. These vibrations are triggered and controlled by Amperfy itself. Some interactions are done by iOS (e.g. long press to show preview), these can't be deactivated. Which interactions are you referring to, that still lead to vibration even when the option is deactivated?

BLeeEZ avatar Jun 21 '24 06:06 BLeeEZ

Sorry for the circumstances!

Tested haptic feedback with both options and didn't recognize a difference. The long press vibration is intended based on system level, understood and familiar from other apps. When pressing the "Play" or "Shuffle" buttons there is no vibration at all, no matter if the option is enabled or disabled.

Some situations where I recognized a vibration with the setting being off:

  • When a song is opened, swiping down the active window from the top to the bottom
  • Clicking on the active song being played in the small bottom bar
  • Enabling / disabling different setting options

In these cases I expected there would be a difference when the vibration setting is disabled. Recognized that because I enabled vibration on a system level and normally like the haptic feedback, using Amperfy feels very haptic to me so I tried to disable the vibrations.

Just a minor issue for sure!

brokoler avatar Jun 21 '24 17:06 brokoler

Hi, I've worked on the feature, when you disable the haptic feedback you disable the vibration when you click a song in the Songs list, or when you click a song in the album view. Indeed the toggle doesn't affect the situations that you have listed. @BLeeEZ should consider if we should expand the toggle functionality for those situations.

danielepintore avatar Jun 24 '24 16:06 danielepintore