search

muak / AiForms.Effects

AiForms.Effects for Xamarin.Forms
MIT License
245 stars 27 forks source link

Add sound effect #10

Closed yuka-abn closed 7 years ago

yuka-abn commented 7 years ago

タップ時、ロングタップ時にシステムサウンドを再生する機能の追加リクエストです。

IsSoundEffectプロパティを追加しTrueの場合にコマンド実行前にシステム音声の再生を行います。

Androidではタップ時になるコントロールと鳴らないコントロールがあるため鳴るコントロールの場合にのみ音声を再生するようにしてます。

muak commented 7 years ago

@yuka-abn なんか凄そうな機能ありがとうございます！ TapGestureRecognizerにCommandが出来てから産廃になりそうでしたが、これでまた存在意義が出てきましたｗ 週末作業します。

muak commented 7 years ago

@yuka-abn すみません。ちょっとお伺いしたいのですが

https://github.com/muak/AiForms.Effects/blob/a51ec798470df6825b08b040ed24ceb6bbc64845/AiForms.Effects.Droid/AddCommandPlatformEffect.cs#L35 https://github.com/muak/AiForms.Effects/blob/a51ec798470df6825b08b040ed24ceb6bbc64845/AiForms.Effects.iOS/AddCommandPlatformEffect.cs#L30

このあたりがpublicなのは何か理由がありますか？ 変更可能なようにですかね？

yuka-abn commented 7 years ago

すいません。他のGithubの通知に流されてて気がつかなかったです。 使う際に調整ができた方が良いと考えたからです。

muak commented 7 years ago

@yuka-abn いえいえ〜 分かりました。 type配列だけprivateにしてしまいましたが、何かのタイミングでpublicに変更しておきます。